Re: [Qemu-devel] [PATCH v2 7/8] block: Reuse success path from bdrv_open()

2014-02-11 Thread Max Reitz
On 10.02.2014 15:56, Kevin Wolf wrote: Am 08.02.2014 um 18:39 hat Max Reitz geschrieben: The fail and success paths of bdrv_file_open() may be further shortened by reusing code already existent in bdrv_open(). This includes bdrv_file_open() not taking the reference to options which allows the re

Re: [Qemu-devel] [PATCH v2 7/8] block: Reuse success path from bdrv_open()

2014-02-10 Thread Benoît Canet
Le Saturday 08 Feb 2014 à 18:39:18 (+0100), Max Reitz a écrit : > The fail and success paths of bdrv_file_open() may be further shortened > by reusing code already existent in bdrv_open(). This includes > bdrv_file_open() not taking the reference to options which allows the > removal of QDECREF(opt

Re: [Qemu-devel] [PATCH v2 7/8] block: Reuse success path from bdrv_open()

2014-02-10 Thread Kevin Wolf
Am 08.02.2014 um 18:39 hat Max Reitz geschrieben: > The fail and success paths of bdrv_file_open() may be further shortened > by reusing code already existent in bdrv_open(). This includes > bdrv_file_open() not taking the reference to options which allows the > removal of QDECREF(options) in that

[Qemu-devel] [PATCH v2 7/8] block: Reuse success path from bdrv_open()

2014-02-08 Thread Max Reitz
The fail and success paths of bdrv_file_open() may be further shortened by reusing code already existent in bdrv_open(). This includes bdrv_file_open() not taking the reference to options which allows the removal of QDECREF(options) in that function. Signed-off-by: Max Reitz --- block.c | 68 +++