Well, programmers have a tendency to use directly the examples that are
given to them (I am one of them). The problem with the cURLpp::Cleanup class
is that it's wrongly used by some programmer each time they need to use
cURLpp in a program. So, I've seen programs with multiple use of that
cleanup class, which is very problematic in multi-threaded code. So, the
thing is that this class is dangerous if misused (which is quite easy to
do), and doesn't give us any real advantage since the initialization of the
libcURL library is done anyway at the first call to curl_easy_perform.
Conclusions: We should just remove this class.
On Sun, Nov 2, 2008 at 2:49 PM, Piotr Dobrogost <curlpp_at_2008.autoera.pl>wrote:
> Jean-Philippe Barette-LaPierre wrote:
> > On Sun, Nov 2, 2008 at 1:29 PM, Piotr Dobrogost <curlpp_at_2008.autoera.pl
> > <mailto:curlpp_at_2008.autoera.pl>> wrote:
> >
> > Jean-Philippe Barette-LaPierre wrote:
> > > I must apologize for this. I am ashamed to say that when I
> > started this
> > > project I was a very junior programmer and I didn't knew about
> > the smart
> > > pointers from the STL. Since then, I never took time to fix that.
> > So, if
> > > you want to change the code where you think it needs to be, don't
> > > hesitate to.
> >
> > Wow, so it's much work waiting to be done.
> > Could you point out other things that need/could be changed for
> > better in curlpp?
> >
> >
> > Hum... There's many things to do, but mostly I just try to fix/implement
> > things that people seems to need. The thing most wanted lately was to
> > have a proper way of building cURLpp on windows, and it seems you handle
> > it well :)
> >
> > - I know I need to cleanup the "cURLpp::Cleanup" object which is just
> > wrong to have. I've seen people misusing it and it cause more trouble
> > than anything.
>
> What's the problem with this?
>
> Regards
> Piotr Dobrogost
> _______________________________________________
> cURLpp mailing list
> cURLpp_at_rrette.com
> http://www.rrette.com/mailman/listinfo/curlpp
>
_______________________________________________
cURLpp mailing list
cURLpp_at_rrette.com
http://www.rrette.com/mailman/listinfo/curlpp
Received on 2008-11-03