Christoph "baka0815" Schwerdtfeger

Added project files for Delphi 13

The rather generically named variable would be IdxChanged, right? This could be named TokenIndexChanged or similar, if you prefer. Are there other things missing or in need of change?

Then this can be merged and closed, right?

Edit: What about moving the whole code block handling the rtComma case to a function that returns True if CheckIndent should be exited? That would allow to remove the additional variable and make the code a bit more readable. (I don't like that huge case statement anyway, but never could be bothered to break it up.) That is something I will look at. Would it be ok, if I change that in the other patch-set ( https://sourceforge.net/p/gexperts/feature-requests/198/ )?

Edit: What about moving the whole code block handling the rtComma case to a function that returns True if CheckIndent should be exited? That would allow to remove the additional variable and make the code a bit more readable. (I don't like that huge case statement anyway, but never could be bothered to break it up.) That is something I will look at. Would it be ok, if I change that in the other patch-set (https://sourceforge.net/p/gexperts/feature-requests/198/)?

The difference is the indentation of the comma. That depends on the settings (see TestDoNotIndentUsesStartingWithComma1Comment vs. TestIndentUsesComment: ExpectedText.Add('uses'); ExpectedText.Add(' Windows // Comment 1'); ExpectedText.Add(' , Graphics // Comment 2'); ExpectedText.Add(' , Forms // Comment 3'); ExpectedText.Add(' ;'); vs. ExpectedText.Add('uses'); ExpectedText.Add(' Windows // Comment 1'); ExpectedText.Add(', Graphics // Comment 2'); ExpectedText.Add(', Forms // Comment 3'); ExpectedText.Add(';');...

Edit: What about moving the whole code block handling the rtComma case to a function that returns True if CheckIndent should be exited? That would allow to remove the additional variable and make the code a bit more readable. (I don't like that huge case statement anyway, but never could be bothered to break it up.) That is something I will look at. Would it be ok, if I change that in the other patch-set (https://sourceforge.net/p/gexperts/feature-requests/198/)?

The difference is the indentation of the comma. That depends on the settings (see TestDoNotIndentUsesStartingWithComma1Comment vs TestIndentUsesComment: ExpectedText.Add('uses'); ExpectedText.Add(' Windows // Comment 1'); ExpectedText.Add(' , Graphics // Comment 2'); ExpectedText.Add(' , Forms // Comment 3'); ExpectedText.Add(' ;'); vs. ExpectedText.Add('uses'); ExpectedText.Add(' Windows // Comment 1'); ExpectedText.Add(', Graphics // Comment 2'); ExpectedText.Add(', Forms // Comment 3'); ExpectedText.Add(';');...