* contrib/ari/gdb_ari.sh (OP eol rule): Also check

	addtion, subtraction, multiplication and division binary operator.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 709a55b9e26f1b79a89ceaee49550c4f87cf78c9..01cefe710f0a981146db7cf62e684101167e4698 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2013-03-20  Pierre Muller  <muller@sourceware.org>
+
+       * contrib/ari/gdb_ari.sh (OP eol rule): Also check
+       addtion, subtraction, multiplication and division binary operator.
+
 2013-03-20  Jan Kratochvil  <jan.kratochvil@redhat.com>
 
        Code cleanup.

diff --git a/gdb/contrib/ari/gdb_ari.sh b/gdb/contrib/ari/gdb_ari.sh
index d5e2a97222bbd0044fe4c0f7c154bfd1930d8fbc..b0fdbf483928dab08f3ab2ec28abb4abde5aae1d 100644 (file)
--- a/gdb/contrib/ari/gdb_ari.sh
+++ b/gdb/contrib/ari/gdb_ari.sh
@@ -609,7 +609,11 @@ BEGIN { doc["OP eol"] = "\
 Do not use &&, or || at the end of a line"
     category["OP eol"] = ari_code
 }
-/(\|\||\&\&|==|!=)[[:space:]]*$/ {
+# * operator needs a special treatment as it can be a
+# valid end of line for a pointer type definition
+# Only catch case where an assignment or an opening brace is present
+/(\|\||\&\&|==|!=|[[:space:]][+\-\/])[[:space:]]*$/ \
+|| /(\(|=)[[:space:]].*[[:space:]]\*[[:space:]]*$/ {
     fail("OP eol")
 }