-
Notifications
You must be signed in to change notification settings - Fork 1k
Compressed chains/fn calls #4146
Copy link
Copy link
Open
Labels
C-feature-requestCategory: a feature request (not decided/implemented)Category: a feature request (not decided/implemented)P-lowLow priorityLow priority
Description
This may be hard to get right, but, in general, I tend to prefer putting two or three things in a fn call instead of putting each on its own line. This prevents things from getting needlessly absurdly vertical, eg this in today's rustfmt:
+ return_monitor_err!(
+ self,
+ e,
+ channel_state,
+ chan,
+ order,
+ revoke_and_ack.is_some(),
+ commitment_update.is_some()
+ );
could be
+ return_monitor_err!(
+ self, e, channel_state,
+ chan, order,
+ revoke_and_ack.is_some(),
+ commitment_update.is_some()
+ );
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C-feature-requestCategory: a feature request (not decided/implemented)Category: a feature request (not decided/implemented)P-lowLow priorityLow priority
Type
Fields
Give feedbackNo fields configured for issues without a type.