On Wed, 4 Mar 2026 14:13:06 GMT, Volkan Yazici <[email protected]> wrote:

>> src/jdk.httpserver/share/classes/sun/net/httpserver/ContextList.java line 
>> 192:
>> 
>>> 190:             if ("/".equals(contextPath)) {
>>> 191:                 return true;
>>> 192:             }
>> 
>> In fact you could probably keep that - and just move it after:
>> 
>>             if (!requestPath.startsWith(contextPath)) {
>>                 return false;
>>             }
>> 
>> below?
>
> AFAICT, the fast path neither brings a significant performance benefit, nor 
> is at a hot spot. Given playing out smart caused this issue, I'd rather keep 
> things simple, and remove the fast path.

OK. Looks good then.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30044#discussion_r2884038188

Reply via email to