Re: ole32: implement IEnumFORMATETC_Next_Proxy (1/2)

[Huw Davies]

On Thu, Oct 23, 2008 at 01:35:55PM -0500, Austin English wrote:
> diff --git a/dlls/ole32/usrmarshal.c b/dlls/ole32/usrmarshal.c
> index b8487c3..be9ebff 100644
> --- a/dlls/ole32/usrmarshal.c
> +++ b/dlls/ole32/usrmarshal.c
> @@ -2398,8 +2398,13 @@ HRESULT CALLBACK IEnumFORMATETC_Next_Proxy(
>      FORMATETC *rgelt,
>      ULONG *pceltFetched)
>  {
> -    FIXME(":stub\n");
> -    return E_NOTIMPL;
> +  ULONG fetched;
> +  if (!pceltFetched)
> +    pceltFetched = &fetched;
> +    return IEnumFORMATETC_RemoteNext_Proxy(This,
> +                       celt,
> +                       rgelt,
> +                       pceltFetched);

There's some strange indentation going on here.  Could you make it all
4-space based?

Thanks,
Huw.
-- 
Huw Davies
huw(a)codeweavers.com

ole32: implement IEnumFORMATETC_Next_Proxy (1/2)

[Austin English]

On Fri, Oct 24, 2008 at 4:36 AM, Huw Davies <huw(a)codeweavers.com> wrote:
> On Thu, Oct 23, 2008 at 01:35:55PM -0500, Austin English wrote:
> > diff --git a/dlls/ole32/usrmarshal.c b/dlls/ole32/usrmarshal.c
> > index b8487c3..be9ebff 100644
> > --- a/dlls/ole32/usrmarshal.c
> > +++ b/dlls/ole32/usrmarshal.c
> > @@ -2398,8 +2398,13 @@ HRESULT CALLBACK IEnumFORMATETC_Next_Proxy(
> >      FORMATETC *rgelt,
> >      ULONG *pceltFetched)
> >  {
> > -    FIXME(":stub\n");
> > -    return E_NOTIMPL;
> > +  ULONG fetched;
> > +  if (!pceltFetched)
> > +    pceltFetched = &fetched;
> > +    return IEnumFORMATETC_RemoteNext_Proxy(This,
> > +                       celt,
> > +                       rgelt,
> > +                       pceltFetched);
>
> There's some strange indentation going on here.  Could you make it all
> 4-space based?
>
> Thanks,
> Huw.
> --
> Huw Davies
> huw(a)codeweavers.com

I thought I had sent the corrected version the first time, but that's
what I get for studying econ while coding. I'm sending the revised
patchset now. Thanks for catching that!

-- 
-Austin