|
| |
| | Source Code Review Guidelines |
 | | Having to figure out how the code is designed, what modules will be compiled in, and other such information, obvious to the developer, will slow the process of code review while we wait for the overview information, and thus, deployment will be delayed. | | | The binaries from this code will be installed as per the install instructions. | | | The code review participants should remain constant from review to review. |
|
http://www.homeport.org/~adam/review.html
(3842 words)
|
|
| |
| | Code Review Protocol |
| | Optional reviewers are selected to participate in the review session because of their knowledge of the subject matter or expertise in the type of code to be reviewed. | | | If the code is complex the leader may choose to have a half hour informational session some time before the actual peer review session where the author of the code provides a high level detail or the code and is available for questions. | | | A Peer Code Review Plan should be developed and documented in the Software Development Plan for a project. |
|
http://usfweb2.usf.edu/ora_ban/pdprocess/codereview/codereviewpolicy.html
(1345 words)
|
|
| |
| | Code Inspection / Review - QA Forums |
| | Reviewers cannot just review a piece of code, they need to know at least what the design is and how it fits into the rest of the system. | | | A good code review checklist should be present in the meeting and should be followed so that a variety of problems are kept in mind and looked for. | | | Reviewing every module may not be cost effective, so review those code modules that will get most benefit from a review. |
|
http://www.qaforums.com/Forum15/HTML/003455.html
(993 words)
|
|
| |
| | A Case for Code Review |
| | The initial review needs to be meaty enough to represent all the complexities of the software component under development. | | | And each experience where code review was part of the development life-cycle, there was always success, of varying degrees to be sure, but each project reaped benefits. | | | The idea behind a good code review process is that the software is exposed. |
|
http://www.gamasutra.com/features/20000314/stenersen_pfv.htm
(5956 words)
|
|
| |
| | Code Review FAQ |
| | A review is focused on a patch's design, implementation, usefulness in fixing a stated problem, and fit within its module. | | | Code review is our basic mechanism for validating the design and implementation of patches. | | | We do want to be continuously upgrading code quality, and making auxiliary changes related to the patch may be the right thing to do. |
|
http://www.mozilla.org/hacking/code-review-faq.html
(2713 words)
|
|
| |
| | Honor Code Review Committee Report |
| | The Honor Code Review Committee proposes that the manner in which first-year students are introduced to the honor code be changed to emphasize its importance. | | | To further stress the honor code as well as to educate students on the difficult topic of plagiarism, the Committee recommends that issues of academic honesty be addressed in all first-year seminars, preferably in a standardized format developed for the seminar program. | | | Because the honor code is the responsibility of both faculty and students, the system works most effectively when both groups are fully committed to it. |
|
http://www.middlebury.edu/academics/honorcode/report.htm
(2445 words)
|
|
| |
| | Security Code Review |
| | When reviewing code, always ask the question, "Is this data validated?" Keep a list of all entry points into your ASP.NET application, such as HTTP headers, query strings, form data, and so on, and make sure that all input is checked for validity at some point. | | | Your code is particularly vulnerable to race conditions if it caches the results of a security check, for example in a static or global variable, and then uses the flag to make subsequent security decisions. | | | Check that your code validates the data type of the data received from posted form fields and other forms of Web input such as query strings. |
|
http://msdn.microsoft.com/library/en-us/secmod/html/secmod94.asp?frame=true
(9682 words)
|
|
| |
| | Smart Bear Software |
| | Whether you're interested in efficient, effective peer code reviews, software metrics reporting, or change analysis, our software will save you time and make your processes more accurate, repeatable, and reportable. | | | CodeReports computes and analyzes source code continuously over time, authors, and projects. | | | "Code Reviewer enables software engineering managers to reduce the amount of time spent on reviewing code resulting in a more streamlined development process." |
|
http://www.codehistorian.com
(212 words)
|
|
| |
| | AEP Practices |
| | When this occurs, groups usually decide that code reviews are not valuable, drop code reviews from their development process, and lose the potential benefits that can stem from a correctly implemented code review. | | | For code reviews to be effective, they have to be fun and creative. | | | When the group members discuss the code, their discussion should focus on important issues such as algorithms, object-oriented programming, and class design. |
|
http://www.parasoft.com/jsp/aep/aep_practices.jsp?practice=CodeReview
(214 words)
|
|
| |
| | PST Code Review Checklist |
| | The success of the review is highly dependent on the code reviewers thoroughness and diligence in reviewing the packet. | | | One of the reasons that we take the time to perform code reviews is to prevent the wrong software solution from being delivered to our users. | | | The code reviewer must produce a list of all major areas that need to be addressed by the developer. |
|
http://www.pst.stsci.edu/guidelines/code_review_guidelines.html
(700 words)
|
|
| |
| | CSDL Code Review Process |
| | That is not the primary goal of the CSDL software review process, which is to create an environment in which participants learn new things about high quality software design and implementation. | | | In addition to listing the files to be reviewed and their location, please provide a list of your "burning questions" regarding this code. | | | In general, help the reviewers to provide you with insights into the most important problems you currently feel you have regarding the code. |
|
http://csdl.ics.hawaii.edu/FAQ/software-review.html
(1215 words)
|
|
| |
| | Michael Swanson's Blog : Code Review and Complexity |
| | You don't want to assign random people to work on a code that requires specific knowledge such as parsers, code generators, compiler code optimizers, numerical methods or engines of games based on strong knowledge of physics (Flight Simulator is a good example). | | | Advocates of XP will often argue that pair-programming eliminates the need for formal code review because the code is reviewed as it is being written. | | | Of course, the primary reason we formally review code is to subjectively comment on other possible ways to accomplish the same functionality, simplify its logic, or identify candidates for |
|
http://blogs.msdn.com/mswanson/articles/154460.aspx
(2148 words)
|
|
| |
| | Code Reviewer |
| | Reviewers usually don't verify that problems were fixed. | | | "Code Reviewer enables software engineering managers to reduce the amount of time spent on reviewing code resulting in a more streamlined development process." | | | Reviews by e-mail or chat aren't much better. |
|
http://www.codehistorian.com/codereviewer-overview.php
(196 words)
|
|
| |
| | PFD: CODE REVIEW COMMITTEE |
| | The City of Phoenix is currently accepting proposed code change submittals to be considered during staff review of the codes. | | | The staff review process is a two-pronged approach. | | | A formal process was recommended to ensure staff receives sufficient input while developing the amendments and administrative processes necessary to effectively implement and apply the building codes and standards. |
|
http://www.ci.phoenix.az.us/fire/fp_adopt.html
(626 words)
|
|
| |
| | Application Security, Code Review |
| | We consider the number of lines of code, language, site complexity, types of interaction with external systems, numbers of roles, type of business, and several other factors in determining an estimate. | | | Many of our customers use us as a part of their development cycle so that our review is an integral part of their software release cycle. | | | We have strict procedures in place for protecting code in our custody and will never disclose it or use it for anything outside the scope of our review. |
|
http://www.aspectsecurity.com/code.html
(504 words)
|
|
| |
| | Mozilla Code Review and the super-reviewer list |
| | The same is true for code that is not currently built with Mozilla, but which will be in the future. | | | Otherwise, there will be a ton of code to review when you're ready, and it could take eons for the review to be completed. | | | Checking into most (but not all) of the Mozilla tree also requires an additional level of pre-check-in code review. |
|
http://www.mozilla.org/hacking/reviewers.html
(2032 words)
|
|
| |
| | C# code review checklist |
| | Discussion of "how" should be in blocks or in-line with the code in question. | | | Is code written against an interface rather than an implementing class? | | | My current client uses PSP extensively, so I've been putting together a checklist for reviewing C# code for use on our new project. |
|
http://weblogs.asp.net/tgraham/archive/2003/12/19/44763.aspx
(981 words)
|
|
| |
| | use Perl Code Review Ladder Mailing List |
| | Simon Cozens writes "I've been threatening to do it for a while, but now I've caved in, and set up the code review ladder mailing list." This list is for reviewing Perl code, and posting Perl code to be reviewed. | | | Basically, instead of being a mailing-list, people could sign up to be mentors (in specific categories if possible, or as code reviewers). | | | Now, if you were going to create something that cool for Perl code reviews I'd be there today. |
|
http://use.perl.org/article.pl?sid=03/09/01/2138209
(828 words)
|
|
| |
| | Code Review |
| | A review of all code related to CompleteSource, focusing on performance, architecture, supportability and maintainability with future versions of CompleteSource. | | | Where to limit columns returned by queries to those required for dependent logic or presentation within the given template. | | | Where to replace Query methods with calls to API methods. |
|
http://www.moai.com/services/code_review.asp
(132 words)
|
|
| |
| | Code Review |
| | These problems should be brought to the attention of the project manager who should redirect efforts back to the Code Development stage. | | | The Code Review activity may be interrupted if: | | | Check that the source code conforms to project coding standard(s). |
|
http://www.cc.gatech.edu/classes/cs8113h_96_spring/process/Code_Review.html
(310 words)
|
|
| |
| | Code Review Terminology |
| | In the shower this morning, it occurred to me it would be useful to write a guide listing certain words and phrases that arise during code reviews and their actual, candid meanings. | | | Review term or phrase Real Meaning --------------------- ------------ I'm not sure this fits in If I have anything to do with our architecture. | | | I think that's against the That's not the way I arrange my coding standards. |
|
http://www.elsop.com/wrc/humor/coderev.htm
(244 words)
|
|
| |
| | Java Code Review: JStyle (Man Machine Systems) |
| | As part of the static analysis that JStyle performs on your Java code, it additionally computes several valuable object-oriented metrics, besides some conventional ones. | | | Metrics such as Depth of Inheritance Tree (DIT), Lack of Cohesion in Methods (LCOM), and Response for Class (RFC) are believed to indicate code complexity from maintenance and reuse angles. | | | is an automated Java code review tool which knows about important Java coding guidelines practiced widely by expert Java developers, and can quickly apply these to any Java code. |
|
http://www.mmsindia.com/jstyle.html
(244 words)
|
|
| |
| | Compuware DevPartner for Visual C++ |
| | With the optional DevPartner Studio Server license, code coverage is collected across distributed machines. | | | With the optional DevPartner Studio Server license, performance data can be collected across the distributed environment, supplying true, end-to-end performance profiling. | | | Users can automatically locate coding errors, measure detailed execution performance and track what lines of code have been executed during the testing of their application. |
|
http://www.compuware.com/products/devpartner/1592_ENG_HTML.htm
(350 words)
|
|
| |
| | DevPartner Studio Professional Edition: Source Code Review |
| | DevPartner Studio Professional Edition source code review comes tightly integrated with the Visual Studio.NET IDE, so you apply the right.NET standards and technologies consistently at every stage of development, even when working with various languages and multi-tiered components. | | | build a model of the entire application to scan different code components in combination and evaluate how effectively.NET features are used. | | | identify and analyze coding errors, inefficient instructions or inappropriate use of one or more languages within a.NET application, making sure code components work correctly separately and together |
|
http://www.compuware.com/products/devpartner/1560_ENG_HTML.htm
(166 words)
|
|
| |
| | Code Review Procedure |
| | The following rating system has been developed to aid in identifying the magnitude of difference: | | | Examine the model codes chapter-by-chapter and compare the model code requirements to current state code requirements, using the current state code as the base document. | | | Report all model code requirements that differ from the current state code and identify any requirement that is not contained in the current state code, following the procedure outlined below. |
|
http://www.sbcc.wa.gov/docs/tagrevew.html
(305 words)
|
|
| |
| | code-review-ladder Info Page |
| | This is a list for posting and reviewing Perl code for style, efficiency, and so on. | | | If someone reviews your code, please review someone else's. | | | To see the collection of prior postings to the list, visit the code-review-ladder Archives. |
|
http://lists.netthink.co.uk/listinfo/code-review-ladder
(265 words)
|
|
| |
| | Building Code Review |
| | The Building Code review is taking place in consultation with industry, government and consumer representatives. | | | The Building Act 2004 brings in the first major review of the New Zealand Building Code since 1991. | | | The review will ensure that performance criteria in the Building Code are in line with the Act, and take into account people’s expectations. |
|
http://www.building.govt.nz/publish/codreview.php
(131 words)
|
|
| |
| | Code Review |
| | Posted @ 1/6/2005 10:31 PM re: Code Review | | | I am looking for checklists and tools regarding code reviews. | | | Posted @ 1/7/2005 12:21 AM Comments on this post are closed |
|
http://weblogs.asp.net/palermo4/archive/2005/01/06/348193.aspx
(37 words)
|
|
| |
| | DaVinci Code Review (logicalrealism.org) |
| | Having just finished it, I'd have to say "not really." It's a good read, reasonably well researched, but not worth all of the fuss it has caused. | | | If nothing else, perhaps 'The DaVinci Code' will make blindly led Christians curious enough, that they will seek out the truth about their belief system! | | | Dan Brown's most recent novel, The DaVinci Code, has gotten a lot of attention of late. |
|
http://www.logicalrealism.org/archives/267
(2004 words)
|
|
|
