core/com.nokia.carbide.cpp.codescanner/src/com/nokia/carbide/cpp/internal/codescanner/messages.properties
author cawthron
Mon, 24 May 2010 15:37:56 -0500
branchC3_BUILDER_WORK
changeset 1393 42f6165e68df
parent 33 2d1c891725ea
permissions -rw-r--r--
write .branch.txt to make the new head
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
2
cawthron
parents:
diff changeset
     1
ErrorParser.ProecessError=Cannot run CodeScanner. Please make sure CodeScanner is on your PATH or go to the Carbide.c++ > CodeScanner Preferences panel and set the CodeScanner Directory.
cawthron
parents:
diff changeset
     2
CSConfigSettings.LoadConfigErrorTitle=Error
cawthron
parents:
diff changeset
     3
CSConfigSettings.LoadConfigErrorMessage=Error reading file 
cawthron
parents:
diff changeset
     4
CSConfigSettings.SaveConfigErrorTitle=Error
cawthron
parents:
diff changeset
     5
CSConfigSettings.SaveConfigErrorMessage=Error saving file 
cawthron
parents:
diff changeset
     6
CSGeneralTabPage.CSDirLabel=CodeScanner directory:
cawthron
parents:
diff changeset
     7
CSGeneralTabPage.CSDirBrowseLabel=Browse...
cawthron
parents:
diff changeset
     8
CSGeneralTabPage.CSDirBrowseMessage=Choose the directory where codescanner.exe lives
cawthron
parents:
diff changeset
     9
CSGeneralTabPage.HTMLResultsLabel=Generate HTML report
cawthron
parents:
diff changeset
    10
CSGeneralTabPage.HTMLResultsMessage=When enabled, output report files in HTML format
cawthron
parents:
diff changeset
    11
CSGeneralTabPage.XMLResultsLabel=Generate XML report
cawthron
parents:
diff changeset
    12
CSGeneralTabPage.XMLResultsMessage=When enabled, output report files in XML format
cawthron
parents:
diff changeset
    13
CSGeneralTabPage.ResultsDirLabel=Reports directory:
cawthron
parents:
diff changeset
    14
CSGeneralTabPage.ResultsDirBrowseLabel=Browse...
cawthron
parents:
diff changeset
    15
CSGeneralTabPage.ResultsDirBrowseMessage=Choose the directory where the generated report files are stored
cawthron
parents:
diff changeset
    16
CSGeneralTabPage.SelectDirTitle=Select a directory...
cawthron
parents:
diff changeset
    17
CSGeneralTabPage.SelectCSDirMessage=Select the location of the CodeScanner command-line tool:
cawthron
parents:
diff changeset
    18
CSGeneralTabPage.SelectResultsDirMessage=Select the output directory for HTML/XML results:
cawthron
parents:
diff changeset
    19
CSGeneralTabPage.AutoscanLabel=Scan project automatically
cawthron
parents:
diff changeset
    20
CSGeneralTabPage.AutoscanMessage=Run CodeScanner on project automatically after each build
cawthron
parents:
diff changeset
    21
CSGeneralTabPage.KbScanLabel=Enable Knowledge Base Scanning
cawthron
parents:
diff changeset
    22
CSGeneralTabPage.KbScanMessage=Scan SDK specific porting issues
cawthron
parents:
diff changeset
    23
CSGeneralTabPage.MissingResultsDirTitle=Missing Reports Directory
cawthron
parents:
diff changeset
    24
CSGeneralTabPage.MissingResultsDirMessage=Please specify a directory for CodeScanner generated reports.
cawthron
parents:
diff changeset
    25
CSPreferencPage.ProjectSettingsLabel=<a href="com.nokia.carbide.cpp.codescanner.ui.CSPropertyPage">Configure Project Specific Settings...</a>
cawthron
parents:
diff changeset
    26
CSPreferencPage.GeneralTabLabel=General
cawthron
parents:
diff changeset
    27
CSPreferencPage.GeneralTabMessage=Configure general CodeScanner options
cawthron
parents:
diff changeset
    28
CSPreferencPage.FileFiltersTabLabel=File Filters
cawthron
parents:
diff changeset
    29
CSPreferencPage.FileFiltersTabMessage=Configure file filters used by CodeScanner
cawthron
parents:
diff changeset
    30
CSPreferencPage.RulesTabLabel=Rules
cawthron
parents:
diff changeset
    31
CSPreferencPage.RulesTabMessage=Configure rules used by CodeScanner
cawthron
parents:
diff changeset
    32
CSPreferencPage.ImportSettingsLabel=Import Settings...
cawthron
parents:
diff changeset
    33
CSPreferencPage.ImportSettingsMessage=Import CodeScanner configuration settings from a file
cawthron
parents:
diff changeset
    34
CSPreferencPage.ExportSettingsLabel=Export Settings...
cawthron
parents:
diff changeset
    35
CSPreferencPage.ExportSettingsMessage=Export CodeScanner configuration settings to a file
cawthron
parents:
diff changeset
    36
CSPreferencPage.ExportDefaultFilename=export_cs_config.xml
cawthron
parents:
diff changeset
    37
CSPreferencPage.SelectImportSettingsMessage=Select a configuration file to import
cawthron
parents:
diff changeset
    38
CSPreferencPage.SelectExportSettingsMessage=Select a configuration file to export
cawthron
parents:
diff changeset
    39
CSPreferencPage.ImportConfigFileErrorTitle=Error
cawthron
parents:
diff changeset
    40
CSPreferencPage.ImportConfigFileErrorMessage=Cannot find the file 
cawthron
parents:
diff changeset
    41
CSPropertyPage.ProjSettingsLabel=Enable Project Specific Settings
cawthron
parents:
diff changeset
    42
CSPropertyPage.WorkspaceSettingslabel=<a href="com.nokia.carbide.cpp.codescanner.ui.CSPreferencePage">Configure Workspace Settings...</a>
cawthron
parents:
diff changeset
    43
CSScanner.ScanningProjectMessage=Running CodeScanner on Project: {0}
cawthron
parents:
diff changeset
    44
CSScanner.ScanningResourceMessage=Running codescanner on \"{0}\".
cawthron
parents:
diff changeset
    45
CSScanner.ProjectInfoErrorMessage=Unable to retrieve project info.
cawthron
parents:
diff changeset
    46
CSScanner.ConfigSettingsErrorMessage=Unable to load CodeScanner settings.
cawthron
parents:
diff changeset
    47
CSScanner.TaskTitle=Running CodeScanner.
cawthron
parents:
diff changeset
    48
CSScanner.ScanCompletedMessage=CodeScanner completed scanning.
cawthron
parents:
diff changeset
    49
FileFiltersTabPage.FileFiltersLabel=Files patterns to filter out
cawthron
parents:
diff changeset
    50
FileFiltersTabPage.AddFileFilterLabel=Add...
cawthron
parents:
diff changeset
    51
FileFiltersTabPage.AddFileFilterMessage=Add a file filter to the list
cawthron
parents:
diff changeset
    52
FileFiltersTabPage.EditFileFilterLabel=Edit...
cawthron
parents:
diff changeset
    53
FileFiltersTabPage.EditFileFilterMessage=Edit selected file filter
cawthron
parents:
diff changeset
    54
FileFiltersTabPage.RemoveFileFilterLabel=Remove
cawthron
parents:
diff changeset
    55
FileFiltersTabPage.RemoveFileFilterMessage=Remove file filter from the list
cawthron
parents:
diff changeset
    56
FileFiltersTabPage.RemoveAllFileFiltersLabel=Remove All
cawthron
parents:
diff changeset
    57
FileFiltersTabPage.RemoveAllFileFiltersMessage=Remove all file filters
cawthron
parents:
diff changeset
    58
FileFiltersTabPage.FileFilterExistTitle=Duplicate File Filter
cawthron
parents:
diff changeset
    59
FileFiltersTabPage.FileFilterExistMessage=File filter already exists
cawthron
parents:
diff changeset
    60
FileFiltersEdit.DialogTitle=Edit File Filter
cawthron
parents:
diff changeset
    61
FileFiltersEdit.EditFileFilterMessage=Enter a regular expression to specify a file filter
cawthron
parents:
diff changeset
    62
FileFiltersEdit.FileFilterLabel=File filter:
cawthron
parents:
diff changeset
    63
KbManager.LoadFailMessage=Unable to load XML file {0} from plugin {1}.
cawthron
parents:
diff changeset
    64
MarkerDetails.CSDialogTitle=CodeScanner information
cawthron
parents:
diff changeset
    65
MarkerDetails.KBDialogTitle=Knowledge base information
cawthron
parents:
diff changeset
    66
MarkerDetails.ReferenceLabel=Reference >>
cawthron
parents:
diff changeset
    67
MarkerDetails.UnknownRuleMessage=Unknown error/warning.
cawthron
parents:
diff changeset
    68
RulesEdit.CategoryTitle=Category:
cawthron
parents:
diff changeset
    69
RulesEdit.SeverityTitle=Severity:
cawthron
parents:
diff changeset
    70
RulesTabPage.RulesTableMessage=CodeScanner rules
cawthron
parents:
diff changeset
    71
RulesTabPage.RulesTableRulesLabel=Rules
cawthron
parents:
diff changeset
    72
RulesTabPage.RulesTableCategoriesLabel=Categories
cawthron
parents:
diff changeset
    73
RulesTabPage.RulesTableSeveritiesLabel=Severities
cawthron
parents:
diff changeset
    74
RulesTabPage.EditForbiddenWordsLabel=Forbidden words :
33
2d1c891725ea Added support for IAD rules; fix for Bug 8251.
stechong
parents: 2
diff changeset
    75
RulesTabPage.EditIconsLabel=Icons:
2
cawthron
parents:
diff changeset
    76
RulesTabPage.EditLFunctionCantLeaveLabel=LFunction ignore :
cawthron
parents:
diff changeset
    77
RulesTabPage.EditLongLinesLabel=Long lines length :
cawthron
parents:
diff changeset
    78
RulesTabPage.EditMissingCClassLabel=C Class ignore :
cawthron
parents:
diff changeset
    79
RulesTabPage.EditOpenLabel=Open() ignore :
cawthron
parents:
diff changeset
    80
RulesTabPage.EditWorryingCommentsLabel=Worrying comments :
cawthron
parents:
diff changeset
    81
RulesTabPage.EditRuleLabel=Edit...
cawthron
parents:
diff changeset
    82
RulesTabPage.EditRuleMessage=Edit selected rule
cawthron
parents:
diff changeset
    83
RulesTabPage.EnableAllRulesLabel=Enable All
cawthron
parents:
diff changeset
    84
RulesTabPage.EnableAllRulesMessage=Enable all the rules
cawthron
parents:
diff changeset
    85
RulesTabPage.DisableAllRulesLabel=Disable All
cawthron
parents:
diff changeset
    86
RulesTabPage.DisableAllRulesMessage=Disable all the rules
cawthron
parents:
diff changeset
    87
RulesTabPage.DetailsLabel=Details:
cawthron
parents:
diff changeset
    88
CSRule.AccessArrayElementWithoutCheck=Whenever an element in an array is accessed using At(), the index should be checked to ensure that it is less than array.Count(). Accessing an invalid index can cause a panic.
cawthron
parents:
diff changeset
    89
CSRule.AccessArrayElementWithoutCheck2=Whenever an element in an array is accessed using [], the index should first be checked to ensure that it is within range. Accessing an invalid index can cause a panic.
cawthron
parents:
diff changeset
    90
CSRule.Activestart=Using CActiveScheduler::Start() can mean that something asynchronous is being made synchronous. Instead, use active objects correctly in an asynchronous way.
cawthron
parents:
diff changeset
    91
CSRule.Activestop=Using CActiveScheduler::Stop() can mean that something asynchronous is being made synchronous. Instead, use active objects correctly in an asynchronous way.
cawthron
parents:
diff changeset
    92
CSRule.Arraypassing=Passing arrays by value is inefficient, causing needless copies and takes up time and memory. Recommend using references instead.
cawthron
parents:
diff changeset
    93
CSRule.Arrayptrcleanup=It is not enough to push a local CArrayPtr class onto the cleanup stack. A TCleanupItem and callback function must be used to avoid leaking the elements.
cawthron
parents:
diff changeset
    94
CSRule.Assertdebuginvariant=Replace __ASSERT_DEBUG(<condition>, User::Invariant()) with ASSERT(<condition>), because it is easier to read.
cawthron
parents:
diff changeset
    95
CSRule.Baddefines=Badly-named definitions using only lowercase letters makes the code harder to maintain and can lead to defects.
cawthron
parents:
diff changeset
    96
CSRule.Baseconstruct=If a leaving method is called before BaseConstructL(), the system can panic as it is trying to clean up an application that has not been fully initialised.
cawthron
parents:
diff changeset
    97
CSRule.CallActiveObjectWithoutCheckingOrStopping=A panic occurs if an active object is started twice. In general, if starting a timer, IsActive() should be called to ensure that the timer is not already running.
cawthron
parents:
diff changeset
    98
CSRule.Changenotification=Using RSAVarChangeNotify to see System Agent changes
cawthron
parents:
diff changeset
    99
CSRule.Cleanup=CleanupStack::Pop() functions should be called with explicit variable parameters to avoid misalignment.
cawthron
parents:
diff changeset
   100
CSRule.Commentcode=Instances of commented out code make the code hard to maintain or interpret clearly, and should be removed.
cawthron
parents:
diff changeset
   101
CSRule.Connect=Ignoring the error value returned by the Connect() function means that an OOM failure or other problem will cause a panic the next time the function is accessed.
cawthron
parents:
diff changeset
   102
CSRule.ConnectAndDontCloseMemberVariable=Calling Connect() or Open() on a member variable without calling Close() in the destructor.
cawthron
parents:
diff changeset
   103
CSRule.Constnames=A badly-named constant will make the source code harder to maintain and make defects more likely.
cawthron
parents:
diff changeset
   104
CSRule.Consttdescptr=Using const descriptor pointer as an argument. For example: use 'const TDesC&' instead of 'const TDesC*'.
cawthron
parents:
diff changeset
   105
CSRule.Controlornull=You should always check the return value of ControlOrNull() for NULL before attempting to access it.
33
2d1c891725ea Added support for IAD rules; fix for Bug 8251.
stechong
parents: 2
diff changeset
   106
CSRule.Crepository=Independent application cannot assume that the Central Repository is set up fully. This means the return value of CRepository::get() cannot be ignored.
2
cawthron
parents:
diff changeset
   107
CSRule.Ctltargettype=The ctl targettype should not be used. Instead, use DLL and explicitly refer to the Control Panel\u2019s DEF file.
33
2d1c891725ea Added support for IAD rules; fix for Bug 8251.
stechong
parents: 2
diff changeset
   108
CSRule.Customizableicons=Due to device customization requirements, independent application must not remove any customization done by the variant team. This means customizable icons cannot be used.
2
cawthron
parents:
diff changeset
   109
CSRule.Debugrom=Debug versions of components in the ROM mean that ROM space is being taken up with debugging information or that logging is still being output. Only release versions should be in the ROM.
cawthron
parents:
diff changeset
   110
CSRule.Declarename=The __DECLARE_NAME macro is historical and serves no purpose anymore and should be removed.
cawthron
parents:
diff changeset
   111
CSRule.DeleteMemberVariable=The Member variable was deleted incorrectly by not assigning it to NULL or another value to prevent accidental access of the deleted object.
cawthron
parents:
diff changeset
   112
CSRule.Destructor=Destructor is accessing/dereferencing data member
cawthron
parents:
diff changeset
   113
CSRule.DoubleSemiColon=Double semicolons at the end of a line are not necessary and can cause a compiler error.
cawthron
parents:
diff changeset
   114
CSRule.Driveletters=Drive letters should not be hard-coded.
cawthron
parents:
diff changeset
   115
CSRule.Eikbuttons=R_EIK_BUTTONS_* resources will not be internationalised, and should not be used. Instead, create your own button resource.
cawthron
parents:
diff changeset
   116
CSRule.Eikonenvstatic=Keep all CEikonEnv::Static() calls to a minimum, because it involves TLS. All apps, controls, and dialogs already contain a pointer to the instance of CEikonEnv as a member variable. If a class does not have access to a CEikonEnv and needs one, it should store it.
cawthron
parents:
diff changeset
   117
CSRule.Enummembers=Badly-named enum members make the code harder to maintain and may cause defects.
cawthron
parents:
diff changeset
   118
CSRule.Enumnames=Badly-named enums make the code harder to maintain and may cause defects.
cawthron
parents:
diff changeset
   119
CSRule.Exportinline=Exporting inline functions can cause those that link to the DLL to fail to build. Exporting functions limits future changes due to considerations of binary-compatibility.
cawthron
parents:
diff changeset
   120
CSRule.Exportpurevirtual=The exporting of pure virtual functions is not recommended by Symbian.
33
2d1c891725ea Added support for IAD rules; fix for Bug 8251.
stechong
parents: 2
diff changeset
   121
CSRule.Flags=Independent application must not use R&D flags nor feature flags via preprocessor statements in the source code. This means bld*.hrh and productvariant.hrh should not be used.
2
cawthron
parents:
diff changeset
   122
CSRule.Foff=_FOFF allows access to data in classes that were not intended for public access. This may cause problems, especially when the location of the data changes.
cawthron
parents:
diff changeset
   123
CSRule.Forbiddenwords=Use of forbidden words in header files, especially those destined for external release. Some words may be forbidden for legal reasons or for platform consistency. Where they exist, alternative allowed words should be used.
cawthron
parents:
diff changeset
   124
CSRule.Forgottoputptroncleanupstack=A memory leak can occur if a variable is not put on the cleanup stack and a leaving function or ELeave is called.
cawthron
parents:
diff changeset
   125
CSRule.Friend=The friend directive is often misused and can indicate problems in the OO design.
cawthron
parents:
diff changeset
   126
CSRule.Goto=Goto should not be used if it can be avoided because it makes the program flow more difficult to follow.
cawthron
parents:
diff changeset
   127
CSRule.Ifassignments=An assignment was found within an If statement, if this was intentionally done separating the assignment from the conditional is still recommended. Also reported when an action statement is on the same line as a conditional check.
cawthron
parents:
diff changeset
   128
CSRule.Ifpreprocessor=\#if in header files should only be used before the main include guards and not around \#include statements or around functional blocks in class definitions.
cawthron
parents:
diff changeset
   129
CSRule.Inheritanceorder=If a C class inherits first from an M class and then a C class, a panic can occur when trying to pop a CBase pointer pointing to such a class from the cleanup stack when in fact a pointer pointing to the first predecessor, the mixin class, was popped instead.
cawthron
parents:
diff changeset
   130
CSRule.Intleaves=Functions that can leave and return a TInt error are problematical for the caller. Functions should do one or the other, but not both.
cawthron
parents:
diff changeset
   131
CSRule.Jmp=Using setjmp and/or longjmp makes code less maintainable.
cawthron
parents:
diff changeset
   132
CSRule.Leave=Leaving function called in non-leaving function.
cawthron
parents:
diff changeset
   133
CSRule.LeaveNoError=Leaving with KErrNone usually indicates a makeshift solution around a design issue rather than a true and proper fix to the architecture.
cawthron
parents:
diff changeset
   134
CSRule.Leavingoperators=Non-leaving functions should not call functions that can leave, like Operator functions.
cawthron
parents:
diff changeset
   135
CSRule.LFunctionCantLeave=A function should not be named with an 'L' if it cannot leave. The only exception is in virtual functions where the function name is defined in the base class so the L cannot be removed. For example, RunL().
cawthron
parents:
diff changeset
   136
CSRule.Longlines=Lines longer than about 100 characters can indicate messy or badly-structured code that is hard to maintain.
cawthron
parents:
diff changeset
   137
CSRule.Magicnumbers=Use of magic numbers, ie, hard-coded in the source, should be converted to named constants to make the code easier to read and maintain.
cawthron
parents:
diff changeset
   138
CSRule.Mclassdestructor=M classes should not contain a destructor.
cawthron
parents:
diff changeset
   139
CSRule.Memberlc=Objects on the cleanup stack (LC method) should not be assigned to member variables.
cawthron
parents:
diff changeset
   140
CSRule.Membervariablecallld=LD functions should not be called on a member variable because ownership can be unclear and may lead to double deletes.
cawthron
parents:
diff changeset
   141
CSRule.Missingcancel=Cancel() should always be called in active object's destructor to cancel an outstanding request if there is one. If there is no request pending then Cancel() just does nothing, but if we do not call Cancel() when having an outstanding request a panic will be raised.
cawthron
parents:
diff changeset
   142
CSRule.Missingcclass=All C classes should inherit from another C class to ensure that all data members are zeroed.
cawthron
parents:
diff changeset
   143
CSRule.Mmpsourcepath=Use of absolute paths in MMP files makes it impossible to relocate the source. Use relative paths instead.
cawthron
parents:
diff changeset
   144
CSRule.Multilangrsc=Using AddResourceFileL() without first using BaflUtils::NearestLanguageFile(), may cause not all language versions of resources to be picked up.
cawthron
parents:
diff changeset
   145
CSRule.Multipledeclarations=Multiple declarations on one line can be confusing. Separate each declaration out into its own separate line.
cawthron
parents:
diff changeset
   146
CSRule.Multipleinheritance=Multiple inheritance from non M-classes
cawthron
parents:
diff changeset
   147
CSRule.Mydocs=The 'mydocs' directory, whose location is subject to change, should not be referenced directly using hard-coded strings.
cawthron
parents:
diff changeset
   148
CSRule.Namespace=Namespaces are often used to work around a poor naming convention.
cawthron
parents:
diff changeset
   149
CSRule.Newlreferences=NewL() and NewLC() functions should return a pointer to an object created on the heap.
cawthron
parents:
diff changeset
   150
CSRule.Noleavetrap=TRAP contains no leaving functions.
cawthron
parents:
diff changeset
   151
CSRule.Nonconsthbufc=HBufC* parameters should almost always be passed as a const pointer.
cawthron
parents:
diff changeset
   152
CSRule.Nonconsttdesc=TDesC& parameters should be passed as a const. If it is not, it may indicate that the coder does not understand descriptors, for example, passing descriptors by value.
cawthron
parents:
diff changeset
   153
CSRule.Nonleavenew=Trapping the 'new(ELeave) CXxx' call is redundant and wasteful as the code to support TRAP is surprisingly large. If the instantiation process really needs not to leave, use 'new CXxx' and check for NULL.
cawthron
parents:
diff changeset
   154
CSRule.Nonunicodeskins=Skin definition files (SKN, SKE) must be in Unicode.
cawthron
parents:
diff changeset
   155
CSRule.Null=There is no need to compare pointer variables to NULL. Use If(ptr) instead.
cawthron
parents:
diff changeset
   156
CSRule.Open=Ignoring the return value from Open() functions (due to OOM, etc.) means that when next the resource is accessed, a panic will result.
cawthron
parents:
diff changeset
   157
CSRule.Pointertoarrays=In C classes, there is no need to use pointers to arrays as data members. Instead, use the arrays themselves to simplify notation, reduce indirection and heap fragmentation.
cawthron
parents:
diff changeset
   158
CSRule.Pragmadisable=Disabling \#pragma warnings can lead to problems because the warnings are probably there for a reason.
cawthron
parents:
diff changeset
   159
CSRule.Pragmamessage=Use of \#pragma messages and warnings during the build stage can interfere with the parsing of the build log.
cawthron
parents:
diff changeset
   160
CSRule.Pragmaother=Only use \#pragma directives on edge cases, like functions consisting of inline assembler without explicit return statements, as their usage can mask valid build warnings and error messages.
cawthron
parents:
diff changeset
   161
CSRule.Privateinheritance=Classes should not be inherited privately. If public or protected inheritance is not appropriate, consider using an amalgamation; that is, have an object of that type as a member variable.
cawthron
parents:
diff changeset
   162
CSRule.Pushaddrvar=If the variable is owned by the code pushing it, it should be stored as a pointer. If it is not owned, the variable address should not be pushed onto the cleanup stack.
cawthron
parents:
diff changeset
   163
CSRule.Pushmember=Data member pushed to cleanup stack can lead to double deletes or memory leakage. Even if no panic can result, it is bad practice and makes maintenance more difficult.
cawthron
parents:
diff changeset
   164
CSRule.Readresource=Using ReadResource() instead of ReadResourceL() can lead to an error scenario where ReadResource() effectively fails silently by buffer overflow.
cawthron
parents:
diff changeset
   165
CSRule.Resourcenotoncleanupstack=If a stack-based resource object is not put on the cleanup stack with CleanupResetAndDestroyPushL() or CleanupClosePushL(), and a leaving function or ELeave is called, a memory leak occurs.
cawthron
parents:
diff changeset
   166
CSRule.Resourcesonheap=There is rarely a real need to put Resource classes on the heap (unless they are not real Resource classes). Doing so can lead to inefficiency and cleanup stack problems.
cawthron
parents:
diff changeset
   167
CSRule.Returndescriptoroutofscope=Returning a TBuf descriptor that is declared locally takes it out of scope. This can cause a crash on WINSCW, although not on WINS.
cawthron
parents:
diff changeset
   168
CSRule.Rfs=Connecting to an RFs is a time-consuming operation (0.1 secs on some devices). Instead, when possible, use the already-connected one in EikonEnv or elsewhere.
cawthron
parents:
diff changeset
   169
CSRule.Rssnames=Resource files (RSS) with clashing NAME fields can cause the wrong resource file to be accessed and lead to incorrect functionality or panics.
cawthron
parents:
diff changeset
   170
CSRule.Stringliterals=_L() string literals should be replaced by the _LIT() macro.
cawthron
parents:
diff changeset
   171
CSRule.Stringsinresourcefiles=Strings should not be defined in RSS or RA files. Instead, they should be put in RLS or other localisable files.
cawthron
parents:
diff changeset
   172
CSRule.Struct=C-style structs should not generally be used. The correct idiom is to use a class with public members.
cawthron
parents:
diff changeset
   173
CSRule.Tcclasses=T classes derived from C classes may have a complex constructors and should to be handled differently. Recomment that you make the T class into a C class, which is easier to maintain.
cawthron
parents:
diff changeset
   174
CSRule.Tclassdestructor=T classes should not contain a destructor.
cawthron
parents:
diff changeset
   175
CSRule.Todocomments="To do" comments in code suggest that it is not finished.
cawthron
parents:
diff changeset
   176
CSRule.Trapcleanup=A TRAP was found leaving something on the cleanup stack. Doing this will cause a panic.
cawthron
parents:
diff changeset
   177
CSRule.Trapeleave=A TRAP is unnecessary if there are no leaving functions.
cawthron
parents:
diff changeset
   178
CSRule.Traprunl=The RunError() function should be used rather than the CActive derivative using its own TRAPD solution within a RunL().
cawthron
parents:
diff changeset
   179
CSRule.Trspassing=TRequestStatus parameters should always be passed by reference not by value. If TRequestStatus is just being used as an error code, then convert it to a TIntPassing.
cawthron
parents:
diff changeset
   180
CSRule.Uids=UIDs must be unique and not duplicated.
cawthron
parents:
diff changeset
   181
CSRule.Uncompressedaif=AIF files should be referenced as 'AIF=' rather than 'data=' or 'file=', otherwise they can bloat the ROM size and slow down application loading.
cawthron
parents:
diff changeset
   182
CSRule.Uncompressedbmp=Using uncompressed bitmaps can significantly bloat the size of ROM images. All occurrences of 'bitmap=' in iby/hby files should be replaced with 'auto-bitmap='. Also, avoid use of 'data=' or 'file=' for bitmaps.
cawthron
parents:
diff changeset
   183
CSRule.Unicodesource=Having Unicode source files (CPP, H, RLS, LOC, RSS, and RA) will break most build systems.
cawthron
parents:
diff changeset
   184
CSRule.Userafter=Generally, User::After() functions are used to skirt timing problems. Typically, they should be removed and the defects fixed properly, by waiting for the correct event to continue execution.
cawthron
parents:
diff changeset
   185
CSRule.Userfree=User::Free() should removed and replaced by explicit deletes, because all objects free their memory on deletion; their destructors are not called and further resources cannot be freed or closed.
cawthron
parents:
diff changeset
   186
CSRule.UserWaitForRequest=User::WaitForRequest() should not be used in UI code because the UI will not respond to redraw events while its thread is stopped.
cawthron
parents:
diff changeset
   187
CSRule.Variablenames=Local variable names should be of the form localVariable and not aLocalVar or iLocalVar. Badly-named variables can be misleading and cause maintenance and coding errors.
cawthron
parents:
diff changeset
   188
CSRule.Voidparameter=Declaring a void parameter is unnecessary. A function declared as DoSomething(void) may as well be declared as DoSomething(). Void casts are also unnecessary.
cawthron
parents:
diff changeset
   189
CSRule.Worryingcomments=The presence of exclamation and question marks in comments indicate something odd about the code or it's unfinished or not fully understood.