Better display of localVariableDeclarationStatement modifiers

[clementdessoude]

Fix #282

@lppedd, this would now be formatted like this:

public boolean localVariableDeclarationWhichBreak() {
    @Nullable
    final BackupStatus lastStatus = BackupStatus.fromDbValue(
      backupRepository.getLastStatus()
    );

    final BackupStatus lastStatus = BackupStatus.fromDbValue(
      backupRepository.getLastStatus()
    );

    @Nullable
    BackupStatus lastStatus = BackupStatus.fromDbValue(
      backupRepository.getLastStatus()
    );

    BackupStatus lastStatus = BackupStatus.fromDbValue(
      backupRepository.getLastStatus()
    );
  }

  public boolean localVariableDeclarationWhichDoNotBreak() {
    @Nullable
    final BackupStatus lastStatus = value;

    final BackupStatus lastStatus = value;

    @Nullable
    BackupStatus lastStatus = value;

    BackupStatus lastStatus = value;
  }

[lppedd]

@clement26695 thank you!
A question: what about this case? How would it become?

public boolean localVariableDeclarationWhichBreak() {
    @Nullable
    final BackupStatus lastStatus = BackupStatus.fromDbValue(      | print width
      backupRepository.aVeryLongFunctionNameWhichGoesOverThePrintWidth(myParam)
    );
    ...

[clementdessoude]

This

class CLassWithEnum {

  public boolean localVariableDeclarationWhichBreak() {
    @Nullable
    final BackupStatus lastStatus = BackupStatus.fromDbValue(backupRepository.aVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryLongFunctionNameWhichGoesOverThePrintWidth(myParam));
    final BackupStatus lastStatus = BackupStatus.fromDbValue(backupRepository.aVeryLongFunctionNameWhichGoesOverThePrintWidth(myParam, myParam, myParam, myParam, myParam, myParam));
  }

}

would be printed like this:

class CLassWithEnum {

  public boolean localVariableDeclarationWhichBreak() {
    @Nullable
    final BackupStatus lastStatus = BackupStatus.fromDbValue(
      backupRepository.aVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryLongFunctionNameWhichGoesOverThePrintWidth(
        myParam
      )
    );
    final BackupStatus lastStatus = BackupStatus.fromDbValue(
      backupRepository.aVeryLongFunctionNameWhichGoesOverThePrintWidth(
        myParam,
        myParam,
        myParam,
        myParam,
        myParam,
        myParam
      )
    );
  }
}

[lppedd]

@clement26695 that's perfect imho.

[lppedd]

@clement26695 morning! Maybe a bit OT, but I wanted to know your opinion.
Take for example this class level constant

                                                                               | print witdh
private static final Logger logger = LoggerFactory.getLogger(BackupApiDelegateImpl.class);

How would you see it formatted? Like the local variable above? Because that's how it is now.

private static final Logger logger = LoggerFactory.getLogger(
  BackupApiDelegateImpl.class
);

Honestly I'd say something like

private static final Logger logger = 
  LoggerFactory.getLogger(BackupApiDelegateImpl.class);

Looks much more clear on the eye.

And in case of nested expressions, it would become

private static final Logger logger = 
  LoggerFactory.getLogger(
    SomethingElse.myExtremelyLongMethodNameWhichGoesOverPrint()
  );

private static final Logger logger = 
  LoggerFactory.getLogger(
    SomethingElse.myExtremelyLongMethodNameWhichGoesOverPrint(
      myParam1,
      myParam2
    )
  );

---

Btw, your changes works fine, I'm already using them.

[clementdessoude]

FYI, I tried to improve this behavior in #255. I tried to be close to the Prettier JS behavior.