On 8/2/12, Kai Tietz <ktiet...@googlemail.com> wrote: > Hi Corinna, > > 2012/8/2 Corinna Vinschen <vinsc...@redhat.com>: >> Hi, >> >> below is about 5% of the entire patch introducing __LONG32. Reviewing >> more than 40000 lines is far from fun, so I made a break at this point. >> Noticable are a couple of places wher I kept long, these are places >> where the definitions are in an !_WIN64 branch. Also, in comutils >> are two inline functions calling Interlocked functions. I changed the >> reinterpret casts for the argument to the Interlocked function to >> LONG *, rather than __LONG32 *, since LONG is the argument type of >> the Win32 function, in prototype and documentation. Hope that's ok. >> >> >> Thanks, >> Corinna > > so patch is ok. I admit that changes of long->__LONG32 in 32-bit case > is of lower interest for us, but well, this is something to be done > for consistency-reasons IMHO. But for now let us concentrate on the > 64-bit case, where this change actual matters for us. > > Thanks, > Kai >
Just curious about how is LONG not suited as replacement but __LONG32? -- O.S. ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
