Re: [PATCH] kernel32: implement GetLogicalProcessorInformation

2011-11-26 Thread GOUJON Alexandre
On 11/25/2011 08:35 PM, Claudio Fontana wrote: That's great! When it's done it will be one less out of tree patch to maintain, always a win. Ciao, C You're free to keep your patches on your own/private tree but if you're discussing here, I guess you wanted your stuff being committed in the ma

Re: [PATCH] kernel32: implement GetLogicalProcessorInformation

2011-11-26 Thread Claudio Fontana
On Fri, Nov 25, 2011 at 8:01 PM, Charles Davis wrote: > > On Nov 25, 2011, at 8:17 AM, Claudio Fontana wrote: > >> On Fri, Nov 25, 2011 at 4:07 PM, Claudio Fontana >> wrote: >>> I have resent the patch as a "try 2" (there was a try 1.5 to fix test >>> issues under windows older than XP SP3). >>>

Re: [PATCH] kernel32: implement GetLogicalProcessorInformation

2011-11-26 Thread Claudio Fontana
On Fri, Nov 25, 2011 at 4:07 PM, Claudio Fontana wrote: > I have resent the patch as a "try 2" (there was a try 1.5 to fix test > issues under windows older than XP SP3). > > The patch contains changes based on your feedback. Which is completely broken, since the U() macro you suggested to use is

Re: [PATCH] kernel32: implement GetLogicalProcessorInformation

2011-11-26 Thread Claudio Fontana
I have resent the patch as a "try 2" (there was a try 1.5 to fix test issues under windows older than XP SP3). The patch contains changes based on your feedback. Some things I stand by, and won't change. If you feel differently, you should argue for their validity a little bit more than "don't us