On Tue, 2010-06-15 at 09:50 -0700, Gilliam, PaulX J wrote: > > -----Original Message----- > > From: linux-nfs-owner@xxxxxxxxxxxxxxx [mailto:linux-nfs-owner@xxxxxxxxxxxxxxx] On Behalf Of Trond Myklebust > > Sent: Monday, June 14, 2010 2:51 PM > > To: linux-nfs@xxxxxxxxxxxxxxx > > Subject: [PATCH 02/15] NFSv4.1: Clean up nfs4_setup_sequence > > > > Firstly, there is little point in first zeroing out the entire struct > > nfs4_sequence_res, and then initialising all fields save one. Just > > initialise the last field to zero... > > The reason one may want to zero out the entire struct is that in the future, someone may add elements to the struct. In that case, > memset(res, 0, sizeof(*res)); > would not have to be changed, and any new elements will "automatically" be initialized to a known value. > > Just a thought. That assumes this is a structure that is likely to change and/or be extended in the future, which is unlikely since the NFSv4.1 protocol specification is complete and the SEQUENCE results are fully contained in the current set of fields. It also assumes that '0' would be a desirable default value for these new fields. Trond -- 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
