On 12.12.2015 00:47, Jon Turney wrote: > On 11/12/2015 10:31, Emil Velikov wrote: >> On 11 December 2015 at 10:20, Martin Peres <[email protected]> wrote: >>> On 11/12/15 11:30, Michel Dänzer wrote: >>>> On 11.12.2015 18:17, Martin Peres wrote: >>>>> On 11/12/15 04:34, Michel Dänzer wrote: >>>>>> From: Adam Jackson <[email protected]> > [...] >>>>>> +#ifndef WIN32 > > Side note: WIN32 just means that the Win32 API is available, and may be > defined even on Cygwin after including windows.h (so it doesn't really > indicate what the target is) > > This is why the circumlocution "!defined(WIN32) || defined(__CYGWIN__)" > (or it's inverse) is used in places, and to be strictly correct should > probably be used here.
Changed accordingly in v3, thanks. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
