[2.9.2] Difference with Prettier for JS

[boris-petrov]

Prettier-Java 2.9.2

Playground link

# Options (if any):
--print-width 100

Input:

var future = recognizer instanceof SpeechRecognizer
  ? speechRecognizer.stopContinuousRecognitionAsync()
  : recognizer.stopContinuousRecognitionAsync();

Output:

var future = recognizer instanceof SpeechRecognizer
	? speechRecognizer.stopContinuousRecognitionAsync()
	: recognizer.stopContinuousRecognitionAsync();

Expected behavior:

var future =
  recognizer instanceof SpeechRecognizer
    ? speechRecognizer.stopContinuousRecognitionAsync()
    : recognizer.stopContinuousRecognitionAsync();

Prettier for JS

[jtkiesel]

Interesting, it seems Prettier JS allows ternaries with "simple" conditionals (like a single boolean variable) to be printed like our plugin does, but not more complex ones. I will have to take a look at their heuristic to replicate their behavior.

[boris-petrov]

I don't think it's worth investing time in this. I'm fine with you just choosing one variant and sticking with it. I don't see a reason to have two different types of printing for the (almost) same type of code. Why complicate things? So for me this issue can be closed. I'll leave the decision to you. :)