On Fri, Mar 24, 2006 at 01:37:54AM +0100, Blaisorblade wrote:
> Yep, it seems it looks good. But 1 bug still and two suggestions.
> See also attached patch (it misses one of the suggestions because it's late 
> and I'm going to sleep).

I merged that patch.

> If possible (there should be such array constructors) this shouldn't be 
> redundant / duplicated with the initial declarations.
> 
> Like write_sigio_fds = (int[]) {-1,-1} and having a common macro like 
> 
> #define SIGIO_FDS_INIT {-1,-1}
> 
> static int write_sigio_fds[2] = SIGIO_FDS_INIT;
> ...
> write_sigio_fds = (int[]) SIGIO_FDS_INIT

I tried this - I couldn't get anything like that to compile.  I always
got an "incompatible types in assignment" error.

Below is the latest version.

				Jeff