Is it getting worse? 8/

[After](https://github.com/arnetheduck/Nim/blob/7717085107b247c7250d138b871240bf5d5c4a0a/compiler/ccgliterals.nim#L62-L74):
    
    
    [
          result,
          rope(s.len),
          makeCString(s),
          rope(
            if isConst:
              "const"
            else:
              ""
            ,
          )
        ],
    
    
    Run

Before:
    
    
    [result, rope(s.len), makeCString(s),
    rope(if isConst: "const" else: "")])
    
    
    Run

I can _kind of_ understand putting each enum element on its own line (makes it 
easy to assign values), but not this.

I use if expressions everywhere instead of temporary variables or to prevent 
introducing mutable variables. It's a single semantic unit and shouldn't be 
broken up like 
[this](https://github.com/arnetheduck/Nim/blob/7717085107b247c7250d138b871240bf5d5c4a0a/compiler/aliases.nim#L136-L145).
 I had to write some lua recently and this kind of format (plus most of things 
being statements, not expressions) got my blood pressure up.

Putting hanging commas inside calls is, idk, preposterous. Hope this is 
unintentional.

Reply via email to