Re: ole32: quiet a ridiculously noisy fixme (resend)

[Huw Davies]

On Wed, Oct 22, 2008 at 08:34:49PM -0500, Austin English wrote:
> Reduces Open Office 3 application's output from infinity to 7 lines.
> Last patch had an unnecessary whitespace change.
>
> -- 
> -Austin
> From 0dead16d2588bce8330ed862f8e5b6d3c6bb71a7 Mon Sep 17 00:00:00 2001
> From: Austin English 
> Date: Wed, 22 Oct 2008 20:28:37 -0500
> Subject: [PATCH] ole32: quiet a ridiculously noisy fixme
>
> dlls/ole32/usrmarshal.c |    6 +++++-
>  1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/dlls/ole32/usrmarshal.c b/dlls/ole32/usrmarshal.c
> index b8487c3..6886dfb 100644
> --- a/dlls/ole32/usrmarshal.c
> +++ b/dlls/ole32/usrmarshal.c
> @@ -2398,7 +2398,11 @@ HRESULT CALLBACK IEnumFORMATETC_Next_Proxy(
>      FORMATETC *rgelt,
>      ULONG *pceltFetched)
>  {
> - FIXME(":stub\n");
> - static int quietfixme = 0;
> - if (quietfixme == 0) {
> -
>    FIXME(":stub\n");
> -
>    quietfixme = 1;
> - }
> return E_NOTIMPL;
>
> }

The real implementation of this would actually result in fewer lines
of code!

See IEnumVARIANT_Next_Proxy / Stub (in oleaut32/usrmarshal.c) for an
example.

Huw.

ole32: quiet a ridiculously noisy fixme (resend)

[Austin English]

On Thu, Oct 23, 2008 at 6:35 AM, Huw Davies  wrote:
> On Wed, Oct 22, 2008 at 08:34:49PM -0500, Austin English wrote:
> > Reduces Open Office 3 application's output from infinity to 7 lines.
> > Last patch had an unnecessary whitespace change.
> >
> > --
> > -Austin
> > From 0dead16d2588bce8330ed862f8e5b6d3c6bb71a7 Mon Sep 17 00:00:00 2001
> > From: Austin English 
> > Date: Wed, 22 Oct 2008 20:28:37 -0500
> > Subject: [PATCH] ole32: quiet a ridiculously noisy fixme
> >
> > dlls/ole32/usrmarshal.c |    6 +++++-
> >  1 files changed, 5 insertions(+), 1 deletions(-)
> >
> > diff --git a/dlls/ole32/usrmarshal.c b/dlls/ole32/usrmarshal.c
> > index b8487c3..6886dfb 100644
> > --- a/dlls/ole32/usrmarshal.c
> > +++ b/dlls/ole32/usrmarshal.c
> > @@ -2398,7 +2398,11 @@ HRESULT CALLBACK IEnumFORMATETC_Next_Proxy(
> >      FORMATETC *rgelt,
> >      ULONG *pceltFetched)
> >  {
> > - FIXME(":stub\n");
> > - static int quietfixme = 0;
> > - if (quietfixme == 0) {
> > -
> >    FIXME(":stub\n");
> > -
> >    quietfixme = 1;
> > - }
> > return E_NOTIMPL;
> >
> > }
>
> The real implementation of this would actually result in fewer lines
> of code!
>
> See IEnumVARIANT_Next_Proxy / Stub (in oleaut32/usrmarshal.c) for an
> example.
> Huw.
>
> Huw Davies
> [e-mail]

Thanks Huw. I implemented IEnumFORMATETC_Next_Proxy, which then got
Open Office complaining about IEnumFORMATETC_Next_Stub. Implementing
that allows it to run!

I'll send to wine-patches shortly.