> My bad, on the array index idiom.  I can redo.

Yes, please do resend without that.

 > With regard to post patch clean-ups, I recall you telling me 
 > that is was preferred to either front-load or back-load the 
 > cleanups in a patch series.  

Yes, that is true.

 > I generally "cleaned-up" the entire functions rather than 
 > just the patched portion.  If I do both together, then you'll 
 > get clean-up noise interspersed with functional deltas making 
 > functional review somewhat annoying in my opinion.

OK, got it.  The changelog "Clean up the SFP+ patch." was misleading.

 - R.
_______________________________________________
ewg mailing list
ewg@lists.openfabrics.org
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg

Reply via email to