Re: [PATCH] ole32: Turn FIXME into a TRACE for clipboard backend of QueryGetData.

[Huw Davies]

On Tue, Sep 10, 2019 at 04:05:42PM +0800, Dmitry Timoshkov wrote:
> This one seems to be implemented.
> Signed-off-by: Dmitry Timoshkov
>
> dlls/ole32/clipboard.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c
> index 3464e2ca6a..0c02d02a65 100644
> --- a/dlls/ole32/clipboard.c
> +++ b/dlls/ole32/clipboard.c
> @@ -1602,7 +1602,7 @@ end:
>   */
>  static HRESULT WINAPI snapshot_QueryGetData(IDataObject *iface, FORMATETC *fmt)
>  {
> - FIXME("(%p, %p {%s})\n", iface, fmt, dump_fmtetc(fmt));
> -
>
> TRACE("(%p, %p {%s})\n", iface, fmt, dump_fmtetc(fmt));
>
> if (!fmt) return E_INVALIDARG;

Yes, the implementation is probably good enough to remove the FIXME
(ideally it would call the data source's own method if there is an
active data source).

Signed-off-by: Huw Davies