forks
/
dfmt
mirror of https://github.com/dlang-community/dfmt.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: boolean logic for line split 

Signed-off-by: Prajwal S N <prajwalnadig21@gmail.com>
This commit is contained in:
Prajwal S N 2024-01-12 13:00:39 +05:30
parent 864caada43
commit b845d96ee2
No known key found for this signature in database
GPG Key ID: 60701A603988FAC2
1 changed files with 2 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/dfmt/ast.d
View File

@ -683,22 +683,10 @@ extern (C++) class FormatVisitor : SemanticTimeTransitiveVisitor
void visitBin(ASTCodegen.BinExp e) void visitBin(ASTCodegen.BinExp e)
{ {
writeExprWithPrecedence(e.e1, precedence[e.op]); writeExprWithPrecedence(e.e1, precedence[e.op]);
if (!config.dfmt_split_operator_at_line_end && e.op in operators && conditionalNewline()) if (!(!config.dfmt_split_operator_at_line_end && e.op in operators && conditionalNewline()))
{
// This block is left empty to ensure conditionalNewline()
// is called and not optimised away by the compiler while
// simplifying boolean expressions.
}
else
write(' '); write(' ');
write(EXPtoString(e.op)); write(EXPtoString(e.op));
if (config.dfmt_split_operator_at_line_end && e.op in operators && conditionalNewline()) if (!(config.dfmt_split_operator_at_line_end && e.op in operators && conditionalNewline()))
{
// This block is left empty to ensure conditionalNewline()
// is called and not optimised away by the compiler while
// simplifying boolean expressions.
}
else
write(' '); write(' ');
writeExprWithPrecedence(e.e2, cast(PREC)(precedence[e.op] + 1)); writeExprWithPrecedence(e.e2, cast(PREC)(precedence[e.op] + 1));
} }