Open main menu

CDOT Wiki β

Changes

JSON Integration Adapter Code Review 1 Changes

495 bytes removed, 14:47, 10 August 2011
General
{| border="1" cellpadding="4" cellspacing="2" style="border: 1px solid black;border-collapse:collapse;"
|-
! Issue! DescriptionTasks
! Status
|-
| Exception TODO| Complete * Create localization strings in en.strings* Use message part as argument to exceptions marked by TODOand not the whole message
|
|-
| Document Documentation* Add documentation to all methods fully in javadoc styleand members| Document every method fully in javadoc style. Also document parameters, and use * Use full English sentences. Make sure special settings like root mode or mode are well documented enough to use
|
|-
| Make sure exceptions output the correct message part.Code Style| Make sure exceptions output the correct message part, for example the inner part, rather than always the root part. More specific is better.* Apply NexJ Code Style| |- | Remove spaces after * Lines that create new Object[]{(remove space here)}Objects should be between newlines| Remove spaces after the curly brackets for all exceptions.| |- | * Add a newline after creating each object or after each decision construct.| Add newlines to conform to NexJ coding style.| |- | Space before break keyword.| Put a space before all break keywords.|
|}
 
===== Design Document =====
1
edit