The Fortran lexer provides both fixed-format and free-format variants. Since DMAP doesn't have a fixed format, all of the code that supports this choice should be removed: isFixFormat should be removed from the function definitions, ColouriseDMAPDocFreeFormat and FoldDMAPDocFreeFormat should be removed, and lmDMAP should refer to ColouriseDMAPDoc and FoldDMAPDoc directly.
scintilla\lexers\LexDMAP.cxx:181: style: The scope of the variable 'chNextNonBlank' can be reduced. Since its not used in classifyFoldPointDMAP chNextNonBlank and the code that sets it can be removed.
IsALineEnd is unused. IsABlank should be declared static.
SCE_DMAP_PREPROCESSOR appears unused.
Its possible but unlikely that DMAP allows old FORTRAN dotted operators like .EQ. and .LT. . Check that you really need SCE_DMAP_OPERATOR2.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
You are, of course, right on all counts. My apologies for the faux pas. I would even go further to suggest the function IsABlank should be removed as it is only used by chNextNonBlank, which is not used, and as a consequence neither is j.
I confirm the defines SCE_DMAP_PREPROCESSOR and SCE_DMAP_OPERATOR2 are not needed either. Please find attached modified source.
I'll look at this after 3.3.6 is released.
The Fortran lexer provides both fixed-format and free-format variants. Since DMAP doesn't have a fixed format, all of the code that supports this choice should be removed: isFixFormat should be removed from the function definitions, ColouriseDMAPDocFreeFormat and FoldDMAPDocFreeFormat should be removed, and lmDMAP should refer to ColouriseDMAPDoc and FoldDMAPDoc directly.
..\lexers\LexDMAP.cxx(50) : warning C4100: 'isFixFormat' : unreferenced formal parameter
..\lexers\LexDMAP.cxx(158) : warning C4100: 'chNextNonBlank' : unreferenced formal parameter
..\lexers\LexDMAP.cxx(169) : warning C4100: 'isFixFormat' : unreferenced formal parameter
scintilla\lexers\LexDMAP.cxx:181: style: The scope of the variable 'chNextNonBlank' can be reduced. Since its not used in classifyFoldPointDMAP chNextNonBlank and the code that sets it can be removed.
IsALineEnd is unused. IsABlank should be declared static.
SCE_DMAP_PREPROCESSOR appears unused.
Its possible but unlikely that DMAP allows old FORTRAN dotted operators like .EQ. and .LT. . Check that you really need SCE_DMAP_OPERATOR2.
Hi Neil,
You are, of course, right on all counts. My apologies for the faux pas. I would even go further to suggest the function IsABlank should be removed as it is only used by chNextNonBlank, which is not used, and as a consequence neither is j.
I confirm the defines SCE_DMAP_PREPROCESSOR and SCE_DMAP_OPERATOR2 are not needed either. Please find attached modified source.
Committed as [ac1e9d].
You should check that lines longer than 72 characters behave as you would expect.
Related
Commit: [ac1e9d]