On 02/02/2009, Oleg Kalnichevski <[email protected]> wrote:
> sebb wrote:
>
> > BasicHttpRequest currently has a non-final field "requestLine".
> >
> > This could be made final, and the check/update of the field in
> > getRequestLine() removed, as the field cannot be null.
> >
> > S///
> >
> >
>
>  Hi Sebastian,
>
>  The reason for this variable being non-final is to make it possible to
> mutate HTTP protocol version of an existing BasicHttpRequest through the
> HttpParams collection associated with the object.

However, there is actually no way to change the private variable -
there is only a getter, and no setter - so other classes cannot change
it.

> It is basically a hangover
> from the 3.x days. I do not mind changing the behavior and making the
> variable final.

Great!

Though in fact it won't change the behaviour, as mentioned above.

Sebb

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to