On Mon, 2012-01-23 at 21:47 -0400, Konrad Rzeszutek Wilk wrote: > On Fri, Jan 20, 2012 at 04:01:12PM -0800, Joe Perches wrote: > > It's equivalent to __printf, so prefer __scanf. > So ... looking at this patch it just seems to macro-fy the > __printf and __scanf attributes. It's just for __scanf. The __printf change is just a neatening/spacing change. > Is this required to make > cleanpatch.pl work easier? No. It's a trivial symmetry patch added to make fewer uses of __attribute__((format(...) similar to the __printf commit from awhile ago. commit b9075fa968a0a4347aef35e235e2995c0e57dddd > And there is also some checkpatch.pl features. Should that part > be in a seperate patch? I think it's OK to do the whole thing at once. _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization