On Thu, Jun 15 2017, Ævar Arnfjörð Bjarmason jotted:
> I'll change it to FREE_AND_NULL and submit my patch as-is, my reading
> of the rest of this thread is that making it a function instead of a
> macro would be interesting, but has its own caveats that are likely
> better considered as part of its own series, whereas this just changes
> existing code to its macro-expanded functional equivalent.
Here's v3 with that change. Nothing but the macro name (and comments,
commit messages etc. referring to it) have changed.
Ævar Arnfjörð Bjarmason (2):
git-compat-util: add a FREE_AND_NULL() wrapper around free(ptr); ptr =
NULL
*.[ch] refactoring: make use of the FREE_AND_NULL() macro
alias.c | 6 ++----
apply.c | 3 +--
attr.c | 6 ++----
blame.c | 3 +--
branch.c | 3 +--
builtin/am.c | 18 +++++-------------
builtin/clean.c | 6 ++----
builtin/config.c | 6 ++----
builtin/index-pack.c | 6 ++----
builtin/pack-objects.c | 12 ++++--------
builtin/unpack-objects.c | 3 +--
builtin/worktree.c | 6 ++----
commit-slab.h | 3 +--
commit.c | 3 +--
config.c | 3 +--
credential.c | 9 +++------
diff-lib.c | 3 +--
diff.c | 6 ++----
diffcore-rename.c | 6 ++----
dir.c | 9 +++------
fast-import.c | 6 ++----
git-compat-util.h | 6 ++++++
gpg-interface.c | 15 +++++----------
grep.c | 12 ++++--------
help.c | 3 +--
http-push.c | 24 ++++++++----------------
http.c | 15 +++++----------
imap-send.c | 3 +--
line-log.c | 6 ++----
ll-merge.c | 3 +--
mailinfo.c | 3 +--
object.c | 3 +--
pathspec.c | 3 +--
prio-queue.c | 3 +--
read-cache.c | 6 ++----
ref-filter.c | 3 +--
refs/files-backend.c | 3 +--
refs/ref-cache.c | 3 +--
remote-testsvn.c | 3 +--
rerere.c | 3 +--
sequencer.c | 3 +--
sha1-array.c | 3 +--
sha1_file.c | 3 +--
split-index.c | 3 +--
transport-helper.c | 27 +++++++++------------------
transport.c | 3 +--
tree-diff.c | 6 ++----
tree-walk.c | 3 +--
tree.c | 3 +--
49 files changed, 103 insertions(+), 197 deletions(-)
--
2.13.1.508.gb3defc5cc