[empathy] Add coding style check for g_hash_table_destroy and g_array_free usage
- From: Xavier Claessens <xclaesse src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [empathy] Add coding style check for g_hash_table_destroy and g_array_free usage
- Date: Wed, 16 Nov 2011 15:05:21 +0000 (UTC)
commit ba6e07cc5a0bf75118c12fd5ebf05e4453a5dc80
Author: Xavier Claessens <xclaesse gmail com>
Date: Wed Nov 16 15:31:19 2011 +0100
Add coding style check for g_hash_table_destroy and g_array_free usage
tools/check-c-style.sh | 13 +++++++++++++
1 files changed, 13 insertions(+), 0 deletions(-)
---
diff --git a/tools/check-c-style.sh b/tools/check-c-style.sh
index dd62fb7..4330b14 100644
--- a/tools/check-c-style.sh
+++ b/tools/check-c-style.sh
@@ -44,6 +44,19 @@ if grep -En '^ *(static |const |)* *[[:alnum:]_]+\*+([[:alnum:]_]|;|$)' \
fail=1
fi
+if grep -n 'g_hash_table_destroy' "$@"; then
+ echo "^^^ Our coding style is to use g_hash_table_unref"
+ fail=1
+fi
+
+for p in "" "ptr_" "byte_"; do
+ if grep -En "g_${p}array_free \(([^ ,]+), TRUE\)" "$@"; then
+ echo "^^^ Our coding style is to use g_${p}array_unref in the case "
+ echo " the underlying C array is not used"
+ fail=1
+ fi
+done
+
if test -n "$CHECK_FOR_LONG_LINES"
then
if egrep -n '.{80,}' "$@"
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]