iamjooon2 commented on PR #652:
URL: https://github.com/apache/tomcat/pull/652#issuecomment-1708120119
Hello from Korea @markt-asf
I unified all of conventions in package apache/coyote
1. unify conditional convention
e.g
```java
// from
if (type == Constants.JK_AJP13_CPING_REQUEST) {
if (protocol.isPaused()) {
recycle();
break;
}
..
}
// to
if (type == Constants.JK_AJP13_CPING_REQUEST && protocol.isPaused()) {
recycle();
break;
}
```
removed unnecessary depth.
2.
unify brackets conventions in apache/coytote
And...
When I first wrote this PR, I wrote it with a pure heart that I also want to
be helpful. I had no other intention
If there's anything lacking on my PR, I'd like you to leave it
Have a good day :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
