On Thu, Jan 20, 2011 at 12:53:01AM +0100, Denys Vlasenko wrote:
> On Wednesday 19 January 2011 20:16, Chris Rees wrote:
> > Hey all,
> > 
> > Still plodding away at updating the FreeBSD port.
> > 
> > Got another patch; FreeBSD etc doesn't have sighandler_t, used in linedit.c.
> > 
> > Perhaps for platform.h?
> 
> Applied, thanks!

Rather than special-casing whether every system has sighandler_t and
possibly breaking things if that changes, why not just do the
following unconditionally?

typedef void (*bb_sighandler_t)(int);

Then use that in place of sighandler_t. Or just write the type out
everywhere it's used. It's not like this type is allowed to differ
between systems.

Rich
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to