[PATCH 1/3] fusion: Implement the IAssemblyEnum interface. [try3]

Jacek Caban jacek at codeweavers.com
Thu Oct 30 07:05:20 CDT 2008


Hi James,

James Hawkins wrote:
> ---
>  dlls/fusion/Makefile.in  |    1 +
>  dlls/fusion/asmcache.c   |   83 ----------
>  dlls/fusion/asmenum.c    |  385 ++++++++++++++++++++++++++++++++++++++++++++++
>  dlls/fusion/asmname.c    |   15 +--
>  dlls/fusion/fusion.c     |   12 --
>  dlls/fusion/fusionpriv.h |   15 ++
>  6 files changed, 402 insertions(+), 109 deletions(-)
>  create mode 100644 dlls/fusion/asmenum.c
>   

+
+done:
+    FindClose(hfind);
+    return S_OK;
+}
+

You always set hr in error paths so I guess you want to return it here.

Also, it's a matter of style, but these gotos look strange. You could 
replace them all by break statements.


Jacek



More information about the wine-devel mailing list