On Fri, 2018-07-13 at 13:40 -0600, Gary Gregory wrote: > On Fri, Jul 13, 2018 at 12:24 PM Oleg Kalnichevski <ol...@apache.org> > wrote: > > > On Fri, 2018-07-13 at 10:27 -0600, Gary Gregory wrote: > > > > > > > ... > > > > > > > > This is now in git master but I think I need another tweak: I > > > made > > > the > > > return type for HttpRequests the class HttpUriRequestBase instead > > > of > > > an > > > interface. I left ClassicHttpRequests to return > > > ClassicHttpRequest > > > but now > > > I wonder if it would not be more flexible to make it return > > > HttpUriRequestBase > > > as well. Thoughts? > > > > > > > Hi Gary > > > > I am sorry it was not quite clear what exactly I had meant. I just > > wanted to have a similar enum applicable to the async methods that > > use > > generic HttpRequest / HttpResponse classes. > > > > I tweaked your code slightly. Please have a look. > > > > Hi Oleg, > > Looks good. > > The remaining question is what should be the return type for > ClassicHttpRequests.create(URI):Â Â ClassicHttpRequest or > HttpUriRequestBase? >
I suppose it should have been HttpUriRequestBase. Corrected. http://git-wip-us.apache.org/repos/asf/httpcomponents-client/commit/0abadca8 Oleg --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org For additional commands, e-mail: dev-h...@hc.apache.org