-
Notifications
You must be signed in to change notification settings - Fork 16
Open
Labels
enhancementNew feature or requestNew feature or request
Milestone
Description
A suggestion from #193 was to remove trailing spaces in conditional directives.
I think this is a good idea, and could be extended to a more general rule for formatting directives.
They can get pretty complicated so we should be careful in implementing this, but there are at least some simple cases that would be nice to cover.
For example
{$ifdef NAME}- normalise casing of the directive (unclear whether lower or upper case is preferable)
- completed in Format line comments and compiler directives #281
- one space between identifier and directive
- no trailing space
- normalise casing of the directive (unclear whether lower or upper case is preferable)
{$COMPILERDIRECTIVE}- no spaces after directive
{$if DEFINED(Foo) and (CONSTANT > 3)}- a real stretch goal would be to format the
{$if}directive expressions like a regular Delphi expression
- a real stretch goal would be to format the
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request