Hello,
Although I like descriptive names, I think that identifiers like "CXTPSyntaxEditLexVariantPtrArrayExAutomatT" used in SyntaxEdit templates are simply way to long.
There are two points against using them:
a) when used in templates they generate Warning 4786 when compiled in VC6 that can only be surpassed using #pragma in stdafx.h (because it bumps up in afxtempl.h):
g:\visualstudio6\vc98\mfc\include\afxtempl.h(63) : warning C4786: 'XTPSyntaxEditLexAnalyser::CXTPSyntaxEditLexObj_SpecCollT<XTPSyntaxEditLexAnalyser::CXTPSyntaxEditLexVariantPtrArrayExAutomatT<&XTPSyntaxEditLexAnalyser::FnName_previous_class>,XTPSyn taxEditLexAnalyser::CXTPSyntaxEditLexVariantPtrArrayExAutomatT<&XTPSyntaxEditLexAnalyser::FnName_previous_tag>,XTPSyntaxEditLexAnalyser::CXTPSyntaxEditLexVariantPtrArrayExAutomatT<&XTPSyntaxEditLexAnalyser::FnName_previous_tag_separators>,XTPSyntaxE ditLexAnalyser::CXTPSyntaxEditLexNULL,XTPSyntaxEditLexAnalyser::CXTPSyntaxEditLexNULL,XTPSyntaxEditLexAnalyser::CXTPSyntaxEditLexNULL,XTPSyntaxEditLexAnalyser::CXTPSyntaxEditLexNULL,XTPSyntaxEditLexAnalyser::CXTPSyntaxEditLexNULL,XTPSyntaxEditLexAna lyser::CXTPSyntaxEditLexNULL,XTPSyntaxEditLexAnalyser::CXTPSyntaxEditLexNULL>' : identifier was truncated to '255' characters in the debug information
b) such long names bump up DLL size significantly as each such name take space in DLL export table. Almost half of megabyte is wasted on such names.
How about shortening:
CXTPSyntaxEdit prefix
to
CXTPSE
for those helper classes?
|