andreas moroder wrote:
> 
> Hello
> 
> would it be possible to add a cast to
> #define smb_buf(buf) (buf + smb_size + CVAL(buf,smb_wct)*2)
> 
> in smb_macros.h
> and change it to ?
> 
> #define smb_buf(buf) (buf + smb_size + (unsigned int ) CVAL(buf,smb_wct)*2)
> 
> This would avoid many warnings from splint and would not change the way the
> macro works.

Last time we did this we got an absolute truckload of extra crud in the
build logs for the IRIX boxes (see build.samba.org) for that exact
reason - the cast doesn't change anything.

I've been meaning to revert these for a while now.  (I hope we can find
some sensible solution that allows both important debuging tools to
work).

Andrew Bartlett

-- 
Andrew Bartlett                                 [EMAIL PROTECTED]
Manager, Authentication Subsystems, Samba Team  [EMAIL PROTECTED]
Student Network Administrator, Hawker College   [EMAIL PROTECTED]
http://samba.org     http://build.samba.org     http://hawkerc.net

Reply via email to