Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jetty-dev] HttpURI

As you correctly worked out in your pull request, these used to be indexes rather than strings and the equality check was really checking if the query was 0 length. 

So definitely a bug

On 23 Sep 2017 01:58, "Guillaume Maillard" <guillaume.maillard@xxxxxxxxx> wrote:
Hi,

Digging in some code I saw in the HttpURI source code :

 public void decodeQueryTo(MultiMap<String> parameters)
    {
        if (_query==_fragment)
            return;


_query and _fragment being defined as String,
I wonder why you are checking strict equality and not using equals.


Regards,

Guillaume

_______________________________________________
jetty-dev mailing list
jetty-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/jetty-dev

Back to the top