[PATCH v2 5/5] include/windows.media.speechrecognition.idl: Add SpeechRecognizer.

Bernhard Kölbl besentv at gmail.com
Tue Mar 1 12:11:04 CST 2022


Imho we should keep it like this because it's easier to compare against the
Windows IDLs.
I mean we also add a bunch of stuff which basically has no effect but keeps
similarity to the original sources, in case MS adds a use to that. Plus,
issues with the namespaces could arise in the future.
(Hope that makes sense.)

Bernhard

Rémi Bernon <rbernon at codeweavers.com> schrieb am Di., 1. März 2022, 18:46:

> On 3/1/22 18:34, Zebediah Figura (she/her) wrote:
> > On 3/1/22 10:42, Rémi Bernon wrote:
> >> From: Bernhard Kölbl <besentv at gmail.com>
> >>
> >> And its dependencies.
> >>
> >> Signed-off-by: Bernhard Kölbl <besentv at gmail.com>
> >> Signed-off-by: Rémi Bernon <rbernon at codeweavers.com>
> >> ---
> >>
> >> v2: Minor alignment tweaking, though how long can these lines be, oof...
> >>
> >
> > Could we omit the fully qualified namespace?
> >
>
> Not for everything, only to reference things in the same namespace.
>
> Although it may seem a good idea, and I initially thought as much,
> there's a lot of cross-namespace references, and I find that it makes
> things less consistent and less readable overall.
>
> --
> Rémi Bernon <rbernon at codeweavers.com>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20220301/b517d9b1/attachment.htm>


More information about the wine-devel mailing list