Bug I found is NOT getting into CVS

Gerald Carter gcarter at valinux.com
Thu Aug 10 14:52:00 GMT 2000


Ron Alexander wrote:
> 
> Funky code
> 
> /* assert macros */
> #define SMB_ASSERT(b) ((b)?(void)0: \
>         (DEBUG(0,("PANIC: assert failed at %s(%d)\n", \
>                  __FILE__, __LINE__)), smb_panic("assert failed")))
> 
> Correct code
> 
> /* assert macros */
> #define SMB_ASSERT(b) ((b)?0: \
>         (DEBUG(0,("PANIC: assert failed at %s(%d)\n", \
>                  __FILE__, __LINE__)), smb_panic("assert failed")))

Ron,

I've made the change in my local CVS check out (just now) and will
check it in with my next update (later today probably).

Anyone have any objections?





Cheers, jerry
-- 
----------------------------------------------------------------------
   /\  Gerald (Jerry) Carter                     Professional Services
 \/    http://www.valinux.com  VA Linux Systems    gcarter at valinux.com
       http://www.samba.org       SAMBA Team           jerry at samba.org
       http://www.eng.auburn.edu/~cartegw

       "...a hundred billion castaways looking for a home."
                                - Sting "Message in a Bottle" ( 1979 )




More information about the samba-technical mailing list