Hmmm. Yes, make sense. That said, I think that it would be better to attempt to do it the right way and then let people tell us when it isn't quite right yet. We should eventually be able to come to a set of changes which are right, for everyone. Compiling on Linux, Solaris, and FreeBSD should be a good start. Perhaps Olga or someone else could help with the Windows compiles. I suspect that we could find someone with access to AIX or HP/UX or ... :-) Perhaps we could also consider cleaning up tests.init? The lists of commented out options seems quite unwieldly to me. Do we really need all of that stuff, especially for the much older systems? Thanx... ps -----Original Message----- From: Tom Haynes [mailto:tdh@xxxxxxxxx] Sent: Wednesday, June 29, 2011 1:29 PM To: Staubach, Peter Cc: tdh@xxxxxxxxx; SteveD@xxxxxxxxxx; linux-nfs@xxxxxxxxxxxxxxx Subject: Re: [PATCH 1/1] Remove warnings on various platforms On Wed, Jun 29, 2011 at 07:53:19AM -0400, peter.staubach@xxxxxxx wrote: > Hi. > > For some of these new casts, wouldn't be it better to simply declare the variable to be the correct type to start with instead of casting some other type? For example, the socklen_t variables? > > It seems funny that all of those XDR routines need to be casted as well. > > Thanx... > > ps I tried to go with the most innocent changes I could. My main concern is lack of access to different platforms to do sanity compiles and unit testing. -- Tom Haynes ex-cfb -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
