Re: Review Request 124856: Call newInstance from the child on first invocation

2015-09-23 Thread Thiago Macieira
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124856/#review85847 --- David: please note that the QtDBus changes made it in for 5.6,

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-09-23 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124856/ --- (Updated Sept. 23, 2015, 7:44 p.m.) Status -- This change has been m

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-09-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124856/#review85325 --- Personnally, I don't like very much the idea of modifying kdel

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-08-21 Thread Thiago Macieira
> On Ago. 21, 2015, 8:08 a.m., David Faure wrote: > > I thought the idea was "there's no race, because as long as the recipient > > doesn't go to the event loop, it won't be processing any incoming messages > > anyway"? > > Albert Astals Cid wrote: > But with Thiago's patches dbus runs in

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-08-21 Thread Albert Astals Cid
> On ago. 21, 2015, 8:08 a.m., David Faure wrote: > > I thought the idea was "there's no race, because as long as the recipient > > doesn't go to the event loop, it won't be processing any incoming messages > > anyway"? > > Albert Astals Cid wrote: > But with Thiago's patches dbus runs in

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-08-21 Thread David Faure
> On Aug. 21, 2015, 8:08 a.m., David Faure wrote: > > I thought the idea was "there's no race, because as long as the recipient > > doesn't go to the event loop, it won't be processing any incoming messages > > anyway"? > > Albert Astals Cid wrote: > But with Thiago's patches dbus runs in

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-08-21 Thread Thiago Macieira
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124856/#review84149 --- Makes sense! - Thiago Macieira On Ago. 21, 2015, 4:06 p.m.,

Re: Review Request 124856: Call newInstance from the child on first invocation

2015-08-21 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124856/ --- (Updated ago. 21, 2015, 4:06 p.m.) Review request for KDE Frameworks and