|
From: | Alberto Garcia |
Subject: | Re: [Qemu-block] [PATCH 02/15] blockjob: Decouple the ID from the device name in the BlockJob struct |
Date: | Tue, 21 Jun 2016 13:42:13 +0200 |
User-agent: | Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Mon 20 Jun 2016 06:17:30 PM CEST, Max Reitz wrote: > I suppose this patch tries to "silently" (i.e. not visibly) introduce > this new ID for now? If so, there is one instance of job->id left that > should probably be changed to job->device (in block_job_complete()). I decided to leave that one unchanged because it's an error message, it doesn't have any other practical impact and it would need to be renamed back to job->id anyway. But now that you mentioned it I see that the error message actually says "...the block job for device '%s'...", so I think I'd rather leave job->id and change the error message instead. Berto
[Prev in Thread] | Current Thread | [Next in Thread] |