From: Chris Koeritz <fred@gruntose.com>
Date: Fri, 22 Feb 2013 18:23:43 +0000 (-0500)
Subject: nice script that looks for mismatches between the logger name
X-Git-Tag: 2.140.90~1094
X-Git-Url: https://feistymeow.org/gitweb/?a=commitdiff_plain;h=455d72525a5bb22da979d8be16a091880c8b5305;p=feisty_meow.git

nice script that looks for mismatches between the logger name
and the class name when creating Log objects with LogFactory.getLog.
---

diff --git a/scripts/buildor/check_java_logger.sh b/scripts/buildor/check_java_logger.sh
new file mode 100644
index 00000000..a343d9fc
--- /dev/null
+++ b/scripts/buildor/check_java_logger.sh
@@ -0,0 +1,33 @@
+#!/bin/bash
+
+# looks for occurrences of getting a Log from the log factory.  when we find
+# one, we make sure it is using the same class name as the compilation unit.
+
+file="$1"; shift
+
+if [ -z "$file" -o ! -f "$file" ]; then
+  echo This script needs a filename to check for appropriate logger creation.
+  echo Any file that has a Log based on a different class than itself will
+  echo be reported.
+  exit 1
+fi
+
+class_in_logger="$(sed -n -e 's/.*LogFactory.getLog( *\([^\.]*\)\.class *).*/\1/p' <"$file")"
+
+#echo got class from logger of $class_in_logger
+
+if [ -z "$class_in_logger" ]; then
+  # we didn't find a log factory.
+  exit 0
+fi
+
+base_of_class="$(basename "$file" | sed -e 's/\(.*\)\.java/\1/')"
+
+#echo base of class is $base_of_class
+
+if [ "$class_in_logger" != "$base_of_class" ]; then
+  echo "$file"
+  exit 1
+fi
+
+