Neels Hofmeyr has uploaded this change for review. (
https://gerrit.osmocom.org/9543
Change subject: verify_value_string_arrays_are_terminated.py: allow terminating
with '{}'
......................................................................
verify_value_string_arrays_are_terminated.py: allow terminating with '{}'
Writing '{ 0, NULL }' is actually identical to just '{}', and that's what I use
these days in all sorts of other contexts. So allow this notation as well in
the C code grepper.
Change-Id: I0822d2d997dccbfb31316953a7b6024c317d92cf
---
M scripts/verify_value_string_arrays_are_terminated.py
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/43/9543/1
diff --git a/scripts/verify_value_string_arrays_are_terminated.py
b/scripts/verify_value_string_arrays_are_terminated.py
index 020bb4d..ad936a5 100755
--- a/scripts/verify_value_string_arrays_are_terminated.py
+++ b/scripts/verify_value_string_arrays_are_terminated.py
@@ -18,7 +18,7 @@
re.MULTILINE | re.DOTALL)
members = r'(\.(value|str)\s*=\s*)?'
-terminator_re = re.compile('{\s*' + members + '(0|NULL)\s*,'
+terminator_re = re.compile('{}|{\s*' + members + '(0|NULL)\s*,'
'\s*' + members + '(0|NULL)\s*}')
errors_found = 0
--
To view, visit https://gerrit.osmocom.org/9543
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0822d2d997dccbfb31316953a7b6024c317d92cf
Gerrit-Change-Number: 9543
Gerrit-PatchSet: 1
Gerrit-Owner: Neels Hofmeyr <nhofm...@sysmocom.de>
