search

opalj / opal

https://www.opal-project.de
Other
51 stars 27 forks source link

Additional operator indentation #188

Open errt opened 10 months ago

errt commented 10 months ago

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 commented 10 months ago

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

maximilianruesch commented 10 months ago

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.