Can't reply on review board, because RB is broken. However I wrote all this review out and I don't want to lose it.<div><br><div><br></div><div><div>Don't do this.</div><div><br></div><div>1) Sync calls are bad</div>
<div><br></div><div>2) Tp has a special way of generating Dbus bindings, where you provide an XML file and it magically builds everything. This already exists in the TpLoggerQt. If you open (in your build folder) _gen/cli-logger.h you will find it actually has these calls.</div>
<div><br></div><div>They should still be wrapped in the LogManager, just like you've done, but instead we should keep an instance of the auto generated LoggerInterface and use that. Also calls should be wrapped in Tp::PendingOperation</div>
<div><br></div><div>so we get something like</div><div><br></div><div>clearAccountHistory(account)</div><div>{</div><div> return Tp::PendingVoid(m_interface->ClearAccount(account->objectPath()), account);</div><div>
}</div><div><br></div><div><br></div><div>For any other project this would be absolutely fine, but TpQt has their own special way, so we should follow that.</div><div><br></div><div>I should have mentioned it in the bug, but I didn't think about it till now.</div>
</div></div>