[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-block] [PATCH v4 08/11] qmp: Add an implementatio
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [Qemu-block] [PATCH v4 08/11] qmp: Add an implementation wrapper for qmp_drive_backup |
Date: |
Mon, 18 May 2015 15:42:07 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, May 11, 2015 at 07:04:23PM -0400, John Snow wrote:
> @@ -2900,9 +2917,16 @@ void qmp_drive_backup(const char *device, const char
> *target,
> }
> }
>
> + /* If we are not supplied with callback override info, use our defaults
> */
> + if (cb == NULL) {
> + cb = block_job_cb;
> + }
> + if (opaque == NULL) {
> + opaque = bs;
> + }
Why assign opaque separately, it raises the question what happens if a
custom cb is given but the caller really wants opaque to be NULL?
The following might be clearer:
if (cb == NULL) {
cb = block_job_cb;
opaque = bs;
}
pgpL53VkR0xYP.pgp
Description: PGP signature
- [Qemu-devel] [PATCH v4 04/11] block: re-add BlkTransactionState, (continued)
- [Qemu-devel] [PATCH v4 04/11] block: re-add BlkTransactionState, John Snow, 2015/05/11
- [Qemu-devel] [PATCH v4 01/11] qapi: Add transaction support to block-dirty-bitmap operations, John Snow, 2015/05/11
- [Qemu-devel] [PATCH v4 06/11] block: add refcount to Job object, John Snow, 2015/05/11
- [Qemu-devel] [PATCH v4 03/11] block: rename BlkTransactionState and BdrvActionOps, John Snow, 2015/05/11
- [Qemu-devel] [PATCH v4 07/11] block: add delayed bitmap successor cleanup, John Snow, 2015/05/11
- [Qemu-devel] [PATCH v4 08/11] qmp: Add an implementation wrapper for qmp_drive_backup, John Snow, 2015/05/11
- Re: [Qemu-devel] [Qemu-block] [PATCH v4 08/11] qmp: Add an implementation wrapper for qmp_drive_backup,
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH v4 05/11] block: add transactional callbacks feature, John Snow, 2015/05/11
- [Qemu-devel] [PATCH v4 10/11] iotests: 124 - transactional failure test, John Snow, 2015/05/11
- [Qemu-devel] [PATCH v4 09/11] block: drive_backup transaction callback support, John Snow, 2015/05/11
- [Qemu-devel] [PATCH v4 11/11] qmp-commands.hx: Update the supported 'transaction' operations, John Snow, 2015/05/11