[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