[PULL] Xserver build fixes for Windows MinGW target.

Jon TURNEY jon.turney at dronecode.org.uk
Wed Nov 28 06:35:53 PST 2012


On 27/11/2012 23:08, Keith Packard wrote:
> Jon TURNEY writes:
> 
>>       os/utils.c: Fix compilation of OsBlockSIGIO when SIGIO isn't defined
> 
> This should probably look like this instead, to avoid compilers
> complaining about unreachable code:
> 
> diff --git a/os/utils.c b/os/utils.c
> index 3c520ad..0d77b84 100644
> --- a/os/utils.c
> +++ b/os/utils.c
> @@ -1227,10 +1227,10 @@ OsBlockSIGIO(void)
>          sigprocmask(SIG_BLOCK, &set, &PreviousSigIOMask);
>          ret = sigismember(&PreviousSigIOMask, SIGIO);
>          return ret;
> -    } else
> -        return 1;
> +    }
>  #endif
>  #endif
> +    return 1;
>  }

Good point.

>> Yaakov Selkowitz (2):
>>       hw/xwin: use raise instead of signal
> 
> This should be fine (it's in POSIX.1-2001 according to the Linux
> raise(3) man page). If we find a system this breaks on, it will be easy
> to fix (testing for raise in the c library and using it only if present).

I'm not quite sure what you are telling me here.

Since this code is in hw/xwin, it's only ever going to be compiled for Windows
targets, anyhow.

Since raise() is in C89, I hope systems that it isn't available on are rare :-)

(Also, it looks like I made a mistake when writing the comment, as it should
say "use raise() instead of kill()" to match the code)

> The other patches were also Reviewed-by: Keith Packard <keithp at keithp.com>

Thanks for taking the time to review these.



More information about the xorg-devel mailing list