Re: [PATCH v3 2/3] ui: Switch "-display sdl" to use the QAPI parser

2022-05-24 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Mon, May 23, 2022 at 09:23:48PM +0200, Thomas Huth wrote: >> On 23/05/2022 15.45, Markus Armbruster wrote: >> > Thomas Huth writes: >> > >> > > The "-display sdl" option still uses a hand-crafted parser for its >> > > parameters since we didn't want to drag an in

Re: [PATCH v3 2/3] ui: Switch "-display sdl" to use the QAPI parser

2022-05-24 Thread Daniel P . Berrangé
On Mon, May 23, 2022 at 09:23:48PM +0200, Thomas Huth wrote: > On 23/05/2022 15.45, Markus Armbruster wrote: > > Thomas Huth writes: > > > > > The "-display sdl" option still uses a hand-crafted parser for its > > > parameters since we didn't want to drag an interface we considered > > > somewhat

Re: [PATCH v3 2/3] ui: Switch "-display sdl" to use the QAPI parser

2022-05-23 Thread Thomas Huth
On 23/05/2022 15.45, Markus Armbruster wrote: Thomas Huth writes: The "-display sdl" option still uses a hand-crafted parser for its parameters since we didn't want to drag an interface we considered somewhat flawed into the QAPI schema. Since the flaws are gone now, it's time to QAPIfy. This

Re: [PATCH v3 2/3] ui: Switch "-display sdl" to use the QAPI parser

2022-05-23 Thread Markus Armbruster
Thomas Huth writes: > The "-display sdl" option still uses a hand-crafted parser for its > parameters since we didn't want to drag an interface we considered > somewhat flawed into the QAPI schema. Since the flaws are gone now, > it's time to QAPIfy. > > This introduces the new "DisplaySDL" QAPI