
15 Apr
2011
15 Apr
'11
10:57 a.m.
Hi Mike,
2011/4/15 Mike Frysinger vapier@gentoo.org:
Do you mean the replacement with space to ident?
According to the 2.6.38.1's checkpatch, it reported: WARNING: please, no spaces at the start of a line
this particular check has already been posted upstream as generally causing more harm than good. but getting checkpatch fixed often feels glacial.
Thanks for your information, I didn't read that article before. Could you provide the URL of related discussion that could be found in the mailing list?
up to Wolfgang how he feels about ifdef indentation -mike
Sure. I think we can discuss about it and then decides how to deal with it. :) Just waiting for Wolfgang's opinion and see if the revised patch is need to be send.
--
Best regards,
Macpaul Lin