[PATCH v5 2/5] windowscodecs: In find_decoder(), get rid of nested if by using goto, and use IWICWineDecoder_Initialize() to initialize decoders.

Ziqing Hui zhui at codeweavers.com
Wed Jul 15 04:04:47 CDT 2020

Signed-off-by: Ziqing Hui <zhui at codeweavers.com>

If we keep the nested if in find_decoder(), after code for calling IWICWineDecoder_Initialize() is added, the nested blocks will become too deep, which makes the code really hard to read. So here I use goto to get rid of them.

v5: It's a new patch in this patch set.

 dlls/windowscodecs/imgfactory.c | 101 ++++++++++++++++----------------
 1 file changed, 50 insertions(+), 51 deletions(-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-windowscodecs-In-find_decoder-get-rid-of-nested-if-b.patch
Type: text/x-patch
Size: 4938 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20200715/74600327/attachment.bin>

More information about the wine-devel mailing list