opal icon indicating copy to clipboard operation
opal copied to clipboard

Additional operator indentation

Open errt opened this issue 1 year ago • 2 comments

As suggested by https://github.com/opalj/opal/pull/187#issuecomment-1904646058

Not sure I like it though, in some places it adds additional indentation to expressions that are already indented a lot. E.g.: https://github.com/opalj/opal/blob/40e1c0adaa29e541359d0941b7782d90b1c9e40f/OPAL/br/src/main/scala/org/opalj/br/analyses/AnalysisApplication.scala#L160C1-L166C56

errt avatar Jan 23 '24 08:01 errt

Note, please treat this as an RFC and comment on whether you are in favor of these changes or not.

errt avatar Jan 23 '24 10:01 errt

I am in favor. The code sample above makes sense, if its too indented in some cases there should always be the option to break the first condition to the next line and reduce indentation that way.

maximilianruesch avatar Jan 23 '24 18:01 maximilianruesch