RFR: JDK-8257401: Use switch expressions in jdk.internal.net.http and java.net.http [v4]

Patrick Concannon pconcannon at openjdk.java.net
Wed Dec 2 16:40:01 UTC 2020


On Wed, 2 Dec 2020 16:28:44 GMT, Kartik Ohri <github.com+27751938+amCap1712 at openjdk.org> wrote:

>> src/java.net.http/share/classes/jdk/internal/net/http/Http1HeaderParser.java line 119:
>> 
>>> 117:         while (canContinueParsing(input)) {
>>> 118:             switch (state) {
>>> 119:                 case INITIAL ->                                     state = State.STATUS_LINE;
>> 
>> Looks good. Although, I think you can improve it further if you align the lambda operators as well
>
> Yes right. Just noticed I had missed that. Fixed in latest commit :)

Looks much better, Kartik. Thanks

-------------

PR: https://git.openjdk.java.net/jdk/pull/1364



More information about the security-dev mailing list