diff --git a/git-hooks/sanitize-commit b/git-hooks/sanitize-commit index d6d4d4482202498f75540a073576f2e30313374d..260c276924943b18b4e8f275d8fcbb33adfa125f 100755 --- a/git-hooks/sanitize-commit +++ b/git-hooks/sanitize-commit @@ -137,7 +137,7 @@ sub complain_spelling($) foreach my $line (@lines) { foreach my $error (@{$errors->{$line}}) { - complain("$line: possible spelling error: $error", "spell"); + complain("$line: Possible spelling error: $error", "spell"); } } } @@ -175,8 +175,8 @@ sub check_apple_terminology() { if ($clike) { if (/\bQ_OS_MAC\b.*&&.*!.*\bQ_OS_IOS\b/) { - complain("$lineno: use of deprecated idiom 'defined(Q_OS_MAC) && !defined(Q_OS_IOS)';" . - "use Q_OS_OSX instead", "terminology"); + complain("$lineno: Use of deprecated idiom 'defined(Q_OS_MAC) && !defined(Q_OS_IOS)';" . + " use Q_OS_OSX instead", "terminology"); } # Not invalid, but remind people about how the unfortunately named Q_OS_MAC must be used @@ -186,24 +186,24 @@ sub check_apple_terminology() } if (/\bQ_OS_MACX\b/) { - complain("$lineno: using deprecated define Q_OS_MACX; use Q_OS_OSX instead", "terminology"); + complain("$lineno: Using deprecated define Q_OS_MACX; use Q_OS_OSX instead", "terminology"); } } elsif ($qmake) { # check qmake scopes if (/\bmac\s*:\s*!\s*ios\b/) { - complain("$lineno: use of deprecated idiom 'mac:!ios', use 'osx' instead", + complain("$lineno: Use of deprecated idiom 'mac:!ios', use 'osx' instead", "terminology"); } # Not invalid, but remind people about how the unfortunately named 'mac' must be used if (/\bmac\b/) { - complain("$lineno: possible use of qmake scope 'mac': this covers both OS X and iOS;" . + complain("$lineno: Possible use of qmake scope 'mac': this covers both OS X and iOS;" . " if you meant only OS X, use 'osx'", "", -1); } # Match the word macx but avoid matching macx- and macx* since these are valid for mkspecs if (/\bmacx\b(?![-*])/) { - complain("$lineno: using deprecated qmake scope 'macx'; use 'osx' instead", + complain("$lineno: Using deprecated qmake scope 'macx'; use 'osx' instead", "terminology"); } } @@ -212,7 +212,7 @@ sub check_apple_terminology() # valid for use, however users are likely to use Mac(intosh)? to incorrectly refer to the OS so # we'll still flag it if (/\bmac(([\s_-]*os)([\s_-]*x)?)?\b/i or /\bmacintosh\b/i) { - complain("$lineno: possible incorrect use of Apple-related terminology", "", -1); + complain("$lineno: Possible incorrect use of Apple-related terminology", "", -1); } }