[GitHub] [tomcat] iamjooon2 commented on pull request #652: Upgrade Readability by removing brackets and depth

2023-09-06 Thread via GitHub
iamjooon2 commented on PR #652: URL: https://github.com/apache/tomcat/pull/652#issuecomment-1708163596 My mistake! I rollbacked some codes. I really appreciate for your comment! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [tomcat] iamjooon2 commented on pull request #652: Upgrade Readability by removing brackets and depth

2023-09-06 Thread via GitHub
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 ==

[GitHub] [tomcat] iamjooon2 commented on pull request #652: Upgrade Readability by removing brackets and depth

2023-09-06 Thread via GitHub
iamjooon2 commented on PR #652: URL: https://github.com/apache/tomcat/pull/652#issuecomment-1708065150 HI @markt-asf I read codes in apache/coyote I found -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub