Re: [Qemu-devel] Missing mon in monitor_cur_is_qmp() and qerror_report()

2010-05-10 Thread Jan Kiszka
Markus Armbruster wrote: > Jan Kiszka writes: > >> Markus Armbruster wrote: >>> Jan Kiszka writes: >>> Markus Armbruster wrote: > Jan Kiszka writes: > >> Luiz, >> >> I missed this when the API was first proposed: >> >> cur_mon is scheduled for removal (one day..

Re: [Qemu-devel] Missing mon in monitor_cur_is_qmp() and qerror_report()

2010-05-10 Thread Markus Armbruster
Jan Kiszka writes: > Markus Armbruster wrote: >> Jan Kiszka writes: >> >>> Markus Armbruster wrote: Jan Kiszka writes: > Luiz, > > I missed this when the API was first proposed: > > cur_mon is scheduled for removal (one day...). It's just an intermediate > ste

Re: [Qemu-devel] Missing mon in monitor_cur_is_qmp() and qerror_report()

2010-05-10 Thread Luiz Capitulino
On Mon, 10 May 2010 14:23:05 +0200 Jan Kiszka wrote: > Markus Armbruster wrote: > > Jan Kiszka writes: > > > >> Markus Armbruster wrote: > >>> Jan Kiszka writes: > >>> > Luiz, > > I missed this when the API was first proposed: > > cur_mon is scheduled for removal (one

Re: [Qemu-devel] Missing mon in monitor_cur_is_qmp() and qerror_report()

2010-05-10 Thread Jan Kiszka
Markus Armbruster wrote: > Jan Kiszka writes: > >> Markus Armbruster wrote: >>> Jan Kiszka writes: >>> Luiz, I missed this when the API was first proposed: cur_mon is scheduled for removal (one day...). It's just an intermediate step to convert all users to explicit

Re: [Qemu-devel] Missing mon in monitor_cur_is_qmp() and qerror_report()

2010-05-10 Thread Markus Armbruster
Jan Kiszka writes: > Markus Armbruster wrote: >> Jan Kiszka writes: >> >>> Luiz, >>> >>> I missed this when the API was first proposed: >>> >>> cur_mon is scheduled for removal (one day...). It's just an intermediate >>> step to convert all users to explicit 'mon' passing. Thus, new APIs >>> sh

Re: [Qemu-devel] Missing mon in monitor_cur_is_qmp() and qerror_report()

2010-05-10 Thread Jan Kiszka
Markus Armbruster wrote: > Jan Kiszka writes: > >> Luiz, >> >> I missed this when the API was first proposed: >> >> cur_mon is scheduled for removal (one day...). It's just an intermediate >> step to convert all users to explicit 'mon' passing. Thus, new APIs >> should not rely it. >> >> I just r

Re: [Qemu-devel] Missing mon in monitor_cur_is_qmp() and qerror_report()

2010-05-10 Thread Markus Armbruster
Jan Kiszka writes: > Luiz, > > I missed this when the API was first proposed: > > cur_mon is scheduled for removal (one day...). It's just an intermediate > step to convert all users to explicit 'mon' passing. Thus, new APIs > should not rely it. > > I just realized that monitor_cur_is_qmp() does

[Qemu-devel] Missing mon in monitor_cur_is_qmp() and qerror_report()

2010-05-07 Thread Jan Kiszka
Luiz, I missed this when the API was first proposed: cur_mon is scheduled for removal (one day...). It's just an intermediate step to convert all users to explicit 'mon' passing. Thus, new APIs should not rely it. I just realized that monitor_cur_is_qmp() does so. It should be refactored to moni