From: Vijay K. <vij...@br...> - 2009-06-14 17:02:01
|
Hi everyone, Michal Simek patched most of the LTP code base to use the Linux kernel coding style. But unfortunately new patches, that are flowing in, seem to add coding style violations. It is better to prevent new style violations than to fix them later on. The kernel developers use a perl script called checkpatch.pl to identify style violations in patches. We could use the same script to check our patches, as well. The script uses files from the kernel to do kernel specific checks. This can be disabled by using the --no-tree option. So to check an LTP patch $ checkpatch.pl --no-tree ltp-patch If everybody is OK with it we can add the script to the LTP source tree, so that everyone checks their patches against the same script. And Subrata can easily check for style violations before merging the patch. In the future, we could also customize the script to do userspace/LTP specific checks, like missing mode argument when open() is invoked with O_CREAT flag. Please do send in your suggestions. Regards, Vijay |