C# Line Delengthening #3776
C# Line Delengthening #3776
Conversation
Sometimes > 120 lines is ok, but the reasons you list are not good ones. The reason to go over 120 lines are:
|
ayoitslilith
force-pushed
the
csharp-line-delengthening
branch
from
18cfce1 to
bd2cf24
Compare
| protected sealed override void QueueTask( | ||
| System.Threading.Tasks.Task task) => execute( | ||
| () => TryExecuteTask(task), | ||
| null); |
There was a problem hiding this comment.
Can we add a using for System.Threading.Tasks and then write
protected sealed override void QueueTask(Task task) =>
execute(() => TryExecuteTask(task), null);
?
There was a problem hiding this comment.
I believe it's already auto-used.
|
|
||
| public void add( | ||
| string name, | ||
| System.Reflection.MethodInfo method, |
There was a problem hiding this comment.
I think adding a using for System.Reflection would be clear to shorten these names, unless it causes a conflict.
| @@ -444,7 +448,8 @@ internal static void doListen(Socket socket, int backlog) | |||
| } | |||
| } | |||
|
|
|||
| internal static int getProtocolSupport(IPAddress addr) => addr.AddressFamily == AddressFamily.InterNetwork ? EnableIPv4 : EnableIPv6; | |||
| internal static int getProtocolSupport(IPAddress addr) => | |||
| addr.AddressFamily == AddressFamily.InterNetwork ? EnableIPv4 : EnableIPv6; | |||
| protected sealed override void QueueTask( | ||
| System.Threading.Tasks.Task task) => execute( | ||
| () => TryExecuteTask(task), | ||
| null); |
There was a problem hiding this comment.
I believe it's already auto-used.
Co-authored-by: Jose <pepone@users.noreply.github.com>
I cannot for the life of me remember why I broke the lines after the return types instead of the lambdas.
| string name, | ||
| System.Reflection.MethodInfo method, | ||
| System.Reflection.PropertyInfo property) => | ||
| _attributes.Add(name, new MemberPropertyResolverI(name, method, property)); |
There was a problem hiding this comment.
No need to indent _attributes here.
| _attributes.Add(name, new MemberPropertyResolverI(name, method, property)); | |
| _attributes.Add(name, new MemberPropertyResolverI(name, method, property)); |
Breaks up lines into more lines because they were all unsightly (and too long for readability!)
Not all lines are broken up. Some I was uncertain about - see here. Others, I made the decision to not break them up for readability's sake. These are <5 characters over the limit (but most often just one) , and the characters that are over the limit are typically of the
)or;varieties. I can go back and list these, however, they will likely be listed with the ones I was uncertain regarding a fix.