<div class="gmail_quote">On Sat, Aug 18, 2012 at 3:54 PM, David Edmundson <span dir="ltr"><<a href="mailto:david@davidedmundson.co.uk" target="_blank">david@davidedmundson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Ok, review time.<br>
<br>
NepomukTpChannelDelegate::NepomukTpChannelDelegate<br>
you don't need use 90% of the factories added.<br>
<br>
NepomukTpChannelDelegate::~NepomukTpChannelDelegate()<br>
d_ptr is leaked<br>
<br>
NepomukTpChannelDelegate::startChat<br>
<br>
Crash if account path isn't correct<br>
Crash if that account isn't online<br>
(possible) Crash if that account is online but contactmanager isn't ready<br>
<br>
<br>
NepomukTpChannelDelegate::finished(Tp::PendingOperation *op)<br>
<br>
This channel hint is no longer correct<br>
Crash if passed contact ID wasn't valid<br></blockquote><div><br></div><div>If we're about to figure out how to start actions on the nepomuk resources (on a general, nepomuk level), this would be obsolete. So the question is - is there going to be such thing?</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
PersonContactModelItem<br>
    void addData(const QUrl &key, const QString &value);<br>
Why is all data strings?<br></blockquote><div><br></div><div>Because there wasn't any need for other types for the basic lib prototype (and is there actually a need for the data not being strings?).</div><div> </div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
PersonsModel<br>
<br>
I don't understand the model structure here, you have a different row<br>
for Email, IM, phone, Mobile Phone, Postal.<br>
Could you explain it to me.<br></blockquote><div><br></div><div>That's how it's supposed to be, each different contact is a different item. You can (and should) filter out those you don't want/need.</div><div>

<br></div></div>-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div>