You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With C# generated code public (static) fields should be pascal cased, not camel cased. An example is lexer generated code:
publicstaticstring[]modeNames={ ...}
should be
publicstaticstring[]ModeNames={ ...}
But then there is a virtual function ModeNames which complicates it. Perhaps modeNames should be a part of the constructor instead as there is no use of it as a virtual function (which also slows down execution) - or perhaps there is an use to override it?
Excerpt from C# identifier naming rules and conventions When naming public members of types, such as fields, properties, events, use pascal casing. Also, use pascal casing for all methods and local functions.
The text was updated successfully, but these errors were encountered:
With C# generated code public (static) fields should be pascal cased, not camel cased. An example is lexer generated code:
should be
But then there is a virtual function
ModeNames
which complicates it. Perhaps modeNames should be a part of the constructor instead as there is no use of it as a virtual function (which also slows down execution) - or perhaps there is an use to override it?Excerpt from C# identifier naming rules and conventions
When naming public members of types, such as fields, properties, events, use pascal casing. Also, use pascal casing for all methods and local functions.
The text was updated successfully, but these errors were encountered: