Fix formatting of comments after statements without semicolons - #311
Conversation
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
609a115 to
fe6aa83
Compare
fourls
left a comment
There was a problem hiding this comment.
Looks good for the most part, but I noticed this behaviour:
begin
if A then
//
begin
// <- should be +1 indent
end
//
;
end;Let's fix + add a test.
I don't think this is related to this change necessarily, but you do also get weird behaviour here:
if A then
//
begin
end;
// <- should be -1 indentWhile it's not grammatically valid (can't have a top level if outside a statement list), it's probably a fixable case of context leakage.
I was able to locate this class of issue and fix it. It also led me to find that anonymous routines weren't being handled correctly. Both of those things should be fixed now, with tests.
I was unable to reproduce this, so I think it is a non-issue with the additional context of it being technically invalid code. |
fourls
left a comment
There was a problem hiding this comment.
Looks good - just one nit.
8aa6cb2 to
1b67893
Compare
1b13dad
into
integrated-application-development:master
This fixes #243.