diff options
author | Jim Meyering <meyering@redhat.com> | 2012-07-11 17:25:27 +0200 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2012-07-12 22:29:26 +0200 |
commit | f344d84d02922ae77f6ec46ebce3bed95dc0f02c (patch) | |
tree | 0fa1b73bd1364878e1220c03d65a826c6f751c0e | |
parent | beee741c19d9287cbabe78a06c458a1d74cffa5d (diff) | |
download | gnulib-f344d84d02922ae77f6ec46ebce3bed95dc0f02c.tar.gz |
maint: enable the sc_avoid_if_before_free syntax-check rule
* cfg.mk (local-checks-to-skip): Enable sc_avoid_if_before_free.
(if_before_free_offenders_): Define.
(if_before_free_basename_re_): Define.
Exempt current files with useless if-before-free.
-rw-r--r-- | ChangeLog | 8 | ||||
-rw-r--r-- | cfg.mk | 20 |
2 files changed, 27 insertions, 1 deletions
@@ -1,3 +1,11 @@ +2012-07-11 Jim Meyering <meyering@redhat.com> + + maint: enable the sc_avoid_if_before_free syntax-check rule + * cfg.mk (local-checks-to-skip): Enable sc_avoid_if_before_free. + (if_before_free_offenders_): Define. + (if_before_free_basename_re_): Define. + Exempt current files with useless if-before-free. + 2012-07-11 Paul Eggert <eggert@cs.ucla.edu> gettext: do not assume '#define ... defined ...' behavior @@ -3,7 +3,6 @@ local-checks-to-skip = \ sc_GFDL_version \ sc_GPL_version \ - sc_avoid_if_before_free \ sc_bindtextdomain \ sc_cast_of_alloca_return_value \ sc_cast_of_argument_to_free \ @@ -59,3 +58,22 @@ exclude_file_name_regexp--sc_trailing_blank = \ # Disable strncpy prohibition completely, for now. exclude_file_name_regexp--sc_prohibit_strncpy = .* + +if_before_free_offenders_ = \ + clean-temp.c \ + csharpcomp.c \ + free.c \ + fstrcmp.c \ + gl_carray_list.c \ + glob.c \ + glthread/lock.c \ + propername.c \ + relocatable.c \ + vasnprintf.c + +if_before_free_basename_re_ = \ + $(shell printf '%s\n' '$(if_before_free_offenders_)' \ + |sed 's|\.c\>||g'|tr -s '[\n[:space:]]' '|') + +exclude_file_name_regexp--sc_avoid_if_before_free = \ + ^lib/($(if_before_free_basename_re_))\.c$$ |