Re: [cpp-netlib-devel] url_encode bug

[Jeroen H. <vex...@gm...>]

On Sun, Jan 10, 2010 at 20:02, Dean Michael Berris
<mik...@gm...> wrote:
> On Mon, Jan 11, 2010 at 2:44 AM, Jeroen Habraken <vex...@gm...> wrote:
>> On Sun, Jan 10, 2010 at 19:29, Dean Michael Berris
>> <mik...@gm...> wrote:
>>>
>>> Thanks, I'm applying and pushing the code as soon as the tests all pass.
>>
>> A checkout of the master seems to pass all tests here on my machine,
>> Linux x86 with GCC 4.4.2.
>>
>
> I meant after I applied the patch. :) It's in branch 0.5-devel now.
>
>>> That reminds me, we should be testing this code especially because we ship it.
>>
>> Very much so, though I'm noticing that compiling the tests is quite
>> resource intensive. I've had GCC eat up more than half of my gigabyte
>> of RAM, with the HTTP 1.1 tests taken more than a couple of minutes.
>>
>
> Yes, I see the same thing.
>
> I'll look at addressing the HTTP 1.1 test issues if I can. The build
> time and resource requirements are really a little disturbing to me,
> but then I think that is the nature of the beast that is C++ and C++
> template metaprogramming. Currently it's not high up on my list
> because I'm concentrating on implementing functionality first rather
> than optimizing the build/test times -- but maybe it has come to the
> point where the limits of GCC is being hit by just the sheer amount of
> code we include when boost/network.hpp is included.

The main reason of starting this discussion was that I was seeing the
same thing happening to my new URI code as it progressed, with the
beautiful but humongous beast that Spirit is.

> It may be time for me to modularize the includes and hopefully I get
> some motivation to do that *soon*. ;)
>
> I'll add issues to the implementation -- if you have any issues and
> feel like logging them in please feel free to do so (and this applies
> to everyone too: http://github.com/mikhailberis/cpp-netlib/issues).
>
> Have a good one and I hope this helps!

You too, and thanks again!

> --
> Dean Michael Berris
> cplusplus-soup.com | twitter.com/deanberris
> linkedin.com/in/mikhailberis | facebook.com/dean.berris | deanberris.com
>

Jeroen