< Previous by Date Date Index Next by Date >
< Previous in Thread Thread Index  

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>