Hi Gili,
This patch looks good. Just one thing i would propose differently: I
think we should mark getBuilder as deprecated and have a new method
getUriBuilder() that we don't break existing code.
Paul.
On Dec 17, 2008, at 8:28 PM, Gili wrote:
>
> I posted a patch here: https://jersey.dev.java.net/issues/show_bug.cgi?id=168
>
> Feel free to take as little or as much of it as you want (that is,
> you can
> ignore changes to WebResource.getBuilder() if you want).
>
> Here are the changes I ended up making:
>
> - WebResource.Builder's constructor no longer takes a URI, it just
> reads the
> value from the enclosing class
> - WebResource.getBuilder() renamed to WebResource.getUriBuilder()
> - Added WebResource.getRequestBuilder()
> - Same for AsyncWebResource
> - Modified two affected tests
>
> Gili
> --
> View this message in context: http://n2.nabble.com/WebResource.getRequestBuilder%28%29-tp1668929p1669358.html
> Sent from the Jersey mailing list archive at Nabble.com.
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe_at_jersey.dev.java.net
> For additional commands, e-mail: users-help_at_jersey.dev.java.net
>