Open ursi opened 8 months ago
That's kinda funny... Your proposed behavior is what I initially implemented. Then, I got suggestions from @Metaxal and @jackfirth to unconditionally enter newline, and I made the switch in https://github.com/sorawee/fmt/commit/831bb4cb6aac95c192be2a9d1b9bcd0f4519b84b#diff-28c7782bc7f8926245fb8e9a590152f17a45b0c9e3dd37d043811d02b8b88476.
I want to note that it is possible for fmt
to use a user-defined style. If you want to follow this approach, I'm happy to guide you through it (or add a support to make this easier). But I really do not want to have a back-and-forth change for the default style without discussion among users. If @Metaxal, @jackfirth, and you have time to discuss with others, I would appreciate that.
I do think the current setting is good. While in principle the two cases are semantically equivalent, I see the first case as a binding between a value and a variable, which should most of the time be a on-liner, while the second case is a function definition, which for consistency should have the body below it.
I think this is a good default, but if this can be made user-adaptive that's even more awesome.
That said, in case no local consensus can be reached, we could still put this to a vote on Discourse (asking for arguments, not just "feelings"), or I would also 100% trust @sorawee to make a reasonable and informed choice.
There could also be a rule about whether to add a newline for the first case, based for example on the number of parentheses in the expression --- say, more than 2 pairs and that's a newline.
I am not a fan of the following formatting:
I would personally like to define functions this small on one line, but I also want to use the shorthand version of
define
. I understand whether or not it should be one line is a matter of taste, but it doesn't make sense to me that the expression that is more noisy is the one that is formatted as one line. To me that feels like a less subjective inconsistency.