svn commit: samba r20938 - in branches/SAMBA_4_0/source/lib/events: .

Volker Lendecke Volker.Lendecke at SerNet.DE
Mon Jan 22 06:04:28 GMT 2007


On Mon, Jan 22, 2007 at 10:35:55AM +1100, tridge at samba.org wrote:
> Volker,
> 
>  > Don't we need sig_atomic_t's here? If I get this right, then
>  > these are variables written by the signal handler and read
>  > by the main program and vice versa.
> 
> Strictly yes, but see:
> 
>   http://www.gnu.org/software/libc/manual/html_node/Atomic-Types.html
> 
> basically 'int' (and uint32_t) is atomic on any platform that Samba
> has any chance of running on. If you want to change it to
> sig_atomic_t and add a configure test then it won't do any harm, but I
> don't think it makes any difference either.

How could using sig_atomic_t hurt here?

Volker
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20070122/c75ad16f/attachment.bin


More information about the samba-technical mailing list