<div dir="ltr"><div><div>Hi Erik,<br></div>something must be wrong if it doesn't compile just because<br>#include <stdlib.h>, anyway i've learnt what is it xfree from this,<br></div><div>thank you.<br><br></div>
<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/8/16 Erik Auerswald <span dir="ltr"><<a href="mailto:auerswal@unix-ag.uni-kl.de" target="_blank">auerswal@unix-ag.uni-kl.de</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello Marcos,<br>
<br>
thanks for your review!<br>
<div class="im"><br>
On Fri, Aug 16, 2013 at 03:25:49PM +0200, Marcos Mayorga wrote:<br>
> what does xfree that doesn't do free? is it NAS a component that require X?<br>
<br>
</div>xfree() is a wrapper around free() that checks for a NULL pointer first.<br>
This is not needed on GNU/Linux.<br>
<div class="im"><br>
> 2013/8/16 Erik Auerswald <<a href="mailto:auerswal@unix-ag.uni-kl.de">auerswal@unix-ag.uni-kl.de</a>><br>
> ><br>
> > the attached patch fixes the following warnings about 'free' when<br>
> > compiling NAS on GNU/Linux x86 (32 bit):<br>
> ><br>
> > ---8<---<br>
> > uevents.c: In function ‘ProcessAudioEvents’:<br>
> > auevents.c:149: warning: incompatible implicit declaration of built-in<br>
> > function ‘free’<br>
> > auevents.c: In function ‘AuDequeueEvents’:<br>
> > auevents.c:249: warning: incompatible implicit declaration of built-in<br>
> > function ‘free’<br>
> > connection.c: In function ‘open_unix_socket’:<br>
> > connection.c:585: warning: incompatible implicit declaration of built-in<br>
> > function ‘free’<br>
> > connection.c:608: warning: incompatible implicit declaration of built-in<br>
> > function ‘free’<br>
> > ---8<---<br>
<br>
</div>In these files, the xfree symbol is defined, but the free symbol is<br>
not. Adding an #include <stdlib.h> resulted in a compile error, thus<br>
the use of xfree.<br>
<br>
Thanks,<br>
Erik<br>
<span class="HOEnZb"><font color="#888888">--<br>
On the internet, nobody can hear you being subtle.<br>
                        -- Linus Torvalds<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>"un dia podré pagarte los servicios, hoy de momento tan solo puedo abonarlos."<br>follow my tweets at <a href="http://twitter.com/mayorga" target="_blank">http://twitter.com/mayorga</a><br>
inspirational stuff<br>
</div>