On Wed, Jun 29, 2011 at 02:16:24PM -0400, peter.staubach@xxxxxxx wrote: > 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. Okay, I'll do that and resubmit. > > 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? Yeah, I've been wondering if modern tools like autoconfig would work here. > > 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 > -- 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
