Re: [PATCH 1/2] include: Add roparameterizediid.idl

[Jacek Caban]

Hi Alex,

On 13.11.2017 06:00, Alex Henrie wrote:
> diff --git a/include/roparameterizediid.idl b/include/roparameterizediid.idl
> new file mode 100644
> index 0000000000..5c3ffd7b5c
> --- /dev/null
> +++ b/include/roparameterizediid.idl

Note that PSDK headers have plain (not IDL-based) header for that. IDL
still may be a good choice for Wine, but in this case you may want to
mark it as winrt with a pragma. See windows.foundation.idl for an example.
> @@ -0,0 +1,91 @@
> +/*
> -
> - Copyright 2017 Alex Henrie
> -
> -
> -
> - This library is free software; you can redistribute it and/or
> -
> - modify it under the terms of the GNU Lesser General Public
> -
> - License as published by the Free Software Foundation; either
> -
> - version 2.1 of the License, or (at your option) any later version.
> -
> -
> -
> - This library is distributed in the hope that it will be useful,
> -
> - but WITHOUT ANY WARRANTY; without even the implied warranty of
> -
> - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> -
> - Lesser General Public License for more details.
> -
> -
> -
> - You should have received a copy of the GNU Lesser General Public
> -
> - License along with this library; if not, write to the Free Software
> -
> - Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
> - */
> -
>
> +import "unknwn.idl";
> -
>
> +typedef void *ROPARAMIIDHANDLE;
> -
>
> +[
> - object
>
> +]
> +interface IRoSimpleMetaDataBuilder : IUnknown

As far as I can see, this shouldn't inherit from IUnknown.

Thanks,

Jacek