Re: [PATCH v2 1/2] git-compat-util: add a FREEZ() wrapper around free(ptr); ptr = NULL

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v2 1/2] git-compat-util: add a FREEZ() wrapper around free(ptr); ptr = NULL
From: Andreas Schwab <schwab@xxxxxxxxxxxxxx>
Date: Sat, 10 Jun 2017 08:55:46 +0200
Cc: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>, git@xxxxxxxxxxxxxxx, Junio C Hamano <gitster@xxxxxxxxx>, Brandon Williams <bmwill@xxxxxxxxxx>, Christian Couder <christian.couder@xxxxxxxxx>, Martin Ågren <martin.agren@xxxxxxxxx>
In-reply-to: <20170609222738.GF21733@aiede.mtv.corp.google.com> (Jonathan Nieder's message of "Fri, 9 Jun 2017 15:27:38 -0700")
References: <20170609085346.19974-1-avarab@gmail.com> <20170609220420.4910-2-avarab@gmail.com> <20170609222738.GF21733@aiede.mtv.corp.google.com>
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)

On Jun 09 2017, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote:

> That way side-effectful callers like FREEZ(func() ? a : b) would
> work.

Except that you cannot take the address of a non-lvalue.

Andreas.

-- 
Andreas Schwab, schwab@xxxxxxxxxxxxxx
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

References:
- [PATCH 0/2] Add a freez() wrapper
  - From: Ævar Arnfjörð Bjarmason
- [PATCH v2 1/2] git-compat-util: add a FREEZ() wrapper around free(ptr); ptr = NULL
  - From: Ævar Arnfjörð Bjarmason
- Re: [PATCH v2 1/2] git-compat-util: add a FREEZ() wrapper around free(ptr); ptr = NULL
  - From: Jonathan Nieder

Prev by Date: Re: [PATCH 2/3] revision.c: use skip_prefix() in handle_revision_opt()
Next by Date: Re: [RFC PATCH 2/4] sha1_file: extract type and size from object_info
Previous by thread: [PATCH v3 2/2] *.[ch] refactoring: make use of the FREE_AND_NULL() macro
Next by thread: [PATCH v2 2/2] *.[ch] refactoring: make use of the FREEZ() macro
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel Development] [Gcc Help] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [V4L] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Fedora Users]