Re: [reSIProcate] resiprocate-1.2 final release
Yup - adding #include <cassert> does the trick under GCC4.2.0. I'll commit
the fix.
Thanks,
Scott
-----Original Message-----
From: Byron Campen [mailto:bcampen@xxxxxxxxxxxx]
Sent: October 29, 2007 11:07 AM
To: Alfred E. Heggestad
Cc: resiprocate-devel; Scott Godin
Subject: Re: [reSIProcate] resiprocate-1.2 final release
Yeah, I saw that patch, but the asserts _really_ do need to be
there.
Scott, can you test whether an #include <cassert> will fix the
problem you were seeing under g++ 4.2?
Best regards,
Byron Campen
> Byron Campen wrote:
>> Hrm. Those asserts really do need to be there though. Try
>> tossing an #include <assert.h> in the header file, and see if that
>> fixes things.
>
> this patch fixes the build for gcc 4.1.2
>
>
> Index: resip/dum/Handle.hxx
> ===================================================================
> --- resip/dum/Handle.hxx (revision 7368)
> +++ resip/dum/Handle.hxx (working copy)
> @@ -2,6 +2,7 @@
> #define RESIP_HANDLE_HXX
>
> #include <iosfwd>
> +#include <assert.h>
> #include "resip/dum/Handled.hxx"
> #include "resip/dum/HandleManager.hxx"
> #include "resip/dum/HandleException.hxx"
>
>
>
> you should review the patch in revision r7310 which fixes the above
> problem by removing the assert's
>
>
>
> /alfred
>
> <snip>