Bug 322771: PostgreSQL incorrectly detects that some summaries have control character...

Patch By Frederic Buclin <LpSolit@gmail.com> r=mkanat, a=justdave

diff --git a/checksetup.pl b/checksetup.pl
index 2cf33249daf7ddbe5734919eca15a075324386b0..2f3307a5ee62cc6d5342d09e6f627c2d66026d32 100755 (executable)
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -4175,17 +4175,22 @@ my $controlchar_bugs =
     $dbh->selectall_arrayref("SELECT short_desc, bug_id FROM bugs WHERE " .
                              "'short_desc' " . $dbh->sql_regexp . 
                              " '[[:cntrl:]]'");
-if (@$controlchar_bugs)
+if (scalar(@$controlchar_bugs))
 {
-    print 'Cleaning control characters from bug summaries...';
+    my $msg = 'Cleaning control characters from bug summaries...';
+    my $found = 0;
     foreach (@$controlchar_bugs) {
         my ($short_desc, $bug_id) = @$_;
-        print " $bug_id...";
-        $short_desc = clean_text($short_desc);
-        $dbh->do("UPDATE bugs SET short_desc = ? WHERE bug_id = ?",
-                 undef, $short_desc, $bug_id);
+        my $clean_short_desc = clean_text($short_desc);
+        if ($clean_short_desc ne $short_desc) {
+            print $msg if !$found;
+            $found = 1;
+            print " $bug_id...";
+            $dbh->do("UPDATE bugs SET short_desc = ? WHERE bug_id = ?",
+                      undef, $clean_short_desc, $bug_id);
+        }
     }
-    print " done.\n";
+    print " done.\n" if $found;
 }
 
 # If you had to change the --TABLE-- definition in any way, then add your