Update tool : checkpatch_tizen_rev2.pl 95/70695/1
authorgs86.lee <gs86.lee@samsung.com>
Fri, 20 May 2016 06:57:28 +0000 (15:57 +0900)
committergs86.lee <gs86.lee@samsung.com>
Fri, 20 May 2016 06:57:28 +0000 (15:57 +0900)
Change-Id: I248664a250923a7a7dee45d244ae159000f5c718

tool/checkpatch_tizen_rev2.pl [moved from tool/checkpatch_tizen.pl with 99% similarity]

similarity index 99%
rename from tool/checkpatch_tizen.pl
rename to tool/checkpatch_tizen_rev2.pl
index 1c29423..2ac085e 100644 (file)
@@ -1512,14 +1512,14 @@ sub process {
                if ($rawline =~ /^\+\s* \t\s*\S/ ||
                    $rawline =~ /^\+\s*        \s*/) {
                        my $herevet = "$here\n" . cat_vet($rawline) . "\n";
-                       WARN("[IDT_R_TAB] code indent should use tabs where possible\n" . $herevet);
+                       ERROR("[IDT_M_TAB] code indent should use tabs where possible\n" . $herevet);
                        $rpt_cleaners = 1;
                }
 
 # check for space before tabs.
                if ($rawline =~ /^\+/ && $rawline =~ / \t/) {
                        my $herevet = "$here\n" . cat_vet($rawline) . "\n";
-                       WARN("[IDT_R_TAB] please, no space before tabs\n" . $herevet);
+                       ERROR("[IDT_M_TAB] please, no space before tabs\n" . $herevet);
                }
 
 # check for spaces at the beginning of a line.
@@ -1529,7 +1529,7 @@ sub process {
 #  3) hanging labels
                if ($rawline =~ /^\+ / && $line !~ /\+ *(?:$;|#|$Ident:)/)  {
                        my $herevet = "$here\n" . cat_vet($rawline) . "\n";
-                       WARN("[IDT_R_TAB] please, no spaces at the start of a line\n" . $herevet);
+                       ERROR("[IDT_M_TAB] please, no spaces at the start of a line\n" . $herevet);
                }
 
 # check we are in a valid C source file if not then ignore this hunk
@@ -1636,7 +1636,7 @@ sub process {
                                }
                        }
                        if ($err ne '') {
-                               WARN("[IDT_R_SCH] switch and case should be at the same indent\n$hereline$err");
+                               ERROR("[IDT_M_SCH] switch and case should be at the same indent\n$hereline$err");
                        }
                }
 
@@ -1821,7 +1821,7 @@ sub process {
 
 # no C99 // comments
                if ($line =~ m{//}) {
-                       WARN("[CMT_M_C89] do not use C99 // comments\n" . $herecurr);
+#                      WARN("[CMT_M_C89] do not use C99 // comments\n" . $herecurr);
                }
                # Remove C99 comments.
                $line =~ s@//.*@@;