On Sat, Feb 07, 2015 at 05:57:46PM +0200, Leon Romanovsky wrote: > Hi Jaegeuk, > > > + /* for batched trimming */ > > + int trim_sections; /* # of sections to trim */ > I would like to suggest to declare trim_sections variable as an > unsigned int and not int, since it can't be negative by definition. > What do you think about it? Hi Leon, No problem. :) I'll change that and merge the patch. Thanks, -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html