Re: [PATCH, v2] xfs: cleanup duplicate initializations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2011-04-14 at 08:53 +1000, Dave Chinner wrote:
> On Wed, Apr 13, 2011 at 05:03:28PM -0500, Alex Elder wrote:
. . .
> > David, perhaps you could sign off on this version.
> > Meanwhile, another reviewer might make sense.
> 
> Seeing as you only added a hunk, I'd say that keeping his old
> sÑgnoff is just fine.

OK.

> > 
> > 					-Alex
> > 
> > follow these guidelines:
> > - leave initialization in the declaration block if it fits the line
> > - move to the code where it's more suitable ('for' init block)
> > 
> > Originally proposed by David Sterba <dsterba@xxxxxxx>
> 
> That is what the "From:" tag is for when you post someone else's
> patch. ;)

Good to know.  Thanks a lot for the review.

> Anyway, looks good now.
> 
> Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
> 
> Cheers,
> 
> Dave.



_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs



[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux