[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v12 0/8] error: auto propagated local_err part I
From: |
Eric Blake |
Subject: |
Re: [PATCH v12 0/8] error: auto propagated local_err part I |
Date: |
Tue, 7 Jul 2020 13:52:43 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0 |
On 7/7/20 11:50 AM, Markus Armbruster wrote:
To speed things up, I'm taking the liberty to respin Vladimir's series
with my documentation amendments.
After my documentation work, I'm very much inclined to rename
ERRP_AUTO_PROPAGATE() to ERRP_GUARD(). The fact that it propagates
below the hood is detail. What matters to its users is that it lets
them use @errp more freely. Thoughts?
I like it - the shorter name is easier to type.
(The rename is a mechanical change, so if we agree to it, we should do
it up front to minimize the churn to all the functions where we add use
of the macro)
Based-on: Message-Id: <20200707160613.848843-1-armbru@redhat.com>
Available from my public repository https://repo.or.cz/qemu/armbru.git
on branch error-auto.
v12: (based on "[PATCH v4 00/45] Less clumsy error checking")
01: Comments merged properly with recent commit "error: Document Error
API usage rules", and edited for clarity. Put ERRP_AUTO_PROPAGATE()
before its helpers, and touch up style.
01-08: Commit messages tweaked
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
- [PATCH v12 4/8] pflash: Use ERRP_AUTO_PROPAGATE(), (continued)
- [PATCH v12 4/8] pflash: Use ERRP_AUTO_PROPAGATE(), Markus Armbruster, 2020/07/07
- [PATCH v12 6/8] virtio-9p: Use ERRP_AUTO_PROPAGATE(), Markus Armbruster, 2020/07/07
- [PATCH v12 2/8] scripts: Coccinelle script to use ERRP_AUTO_PROPAGATE(), Markus Armbruster, 2020/07/07
- [PATCH v12 8/8] xen: Use ERRP_AUTO_PROPAGATE(), Markus Armbruster, 2020/07/07
- [PATCH v12 5/8] fw_cfg: Use ERRP_AUTO_PROPAGATE(), Markus Armbruster, 2020/07/07
- [PATCH v12 7/8] nbd: Use ERRP_AUTO_PROPAGATE(), Markus Armbruster, 2020/07/07
- Re: [PATCH v12 0/8] error: auto propagated local_err part I, Vladimir Sementsov-Ogievskiy, 2020/07/07
- Re: [PATCH v12 0/8] error: auto propagated local_err part I,
Eric Blake <=