<div dir="ltr"><div>Hi All,<br>I've applied all your suggestions and committed them to the repository. [1]<br><br></div>If there are any suggestions or issue, I'll be glad to solve them :)<br><div><br></div><div>Cheers<br>
</div><div><br>[1] <a href="https://github.com/R4md4c/image-sharer/commit/b48a14e58415779f824d37afad069f44749e751a">https://github.com/R4md4c/image-sharer/commit/b48a14e58415779f824d37afad069f44749e751a</a><br></div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 7, 2014 at 10:51 AM, Martin Klapetek <span dir="ltr"><<a href="mailto:martin.klapetek@gmail.com" target="_blank">martin.klapetek@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hey,<div class="gmail_extra"><br><div class="gmail_quote"><div class="">On Mon, Apr 7, 2014 at 3:28 AM, Ahmed I. Khalil <span dir="ltr"><<a href="mailto:ahmedibrahimkhali@gmail.com" target="_blank">ahmedibrahimkhali@gmail.com</a>></span> wrote:<br>


</div><div class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi,<br>
<br>
I've written this library to add this feature [1] to KTp.<br>
The library could also be used or linked with other different KTp<br>
components.<br>
<br>
You can find the library code on Github [2]<br></blockquote><div><br></div></div><div>Damn good job :)<br></div><div class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">



<br>
I'd like someone from KTp development team(David maybe?) to take a<br>
look at the code and tell me about any shortcomings about it.<br></blockquote><div><br></div></div><div>Quickly looking over ShareProvider, the slot name "finishedContentData" is a weird naming, you can use "finishedContentReading" or "finishedFileReading" or something. Also you randomly mix the slots naming, we try to stick to one naming scheme like "onSomethingHappened", "finishedDoingSomething" or "onSomethingFinished", so you might watch that out.</div>


<div><br></div><div>In imgur sharer you do "qDebug() << responseData;" -- which is perfectly fine for local debug, but watch out what you put in the final code as we don't want to make the output too noisy.</div>


<div><br></div><div>Some documentation in the abstract sharer .h would be awesome.</div><div><br></div><div>License - for libraries it's better to use LGPL (v2.1) rather than GPL.</div><div><br></div><div>Thanks for working on this!</div>


<div><br></div><div>Cheers</div></div><span class="HOEnZb"><font color="#888888">-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div>
</font></span></div></div>
<br>_______________________________________________<br>
KDE-Telepathy mailing list<br>
<a href="mailto:KDE-Telepathy@kde.org">KDE-Telepathy@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-telepathy" target="_blank">https://mail.kde.org/mailman/listinfo/kde-telepathy</a><br>
<br></blockquote></div><br></div>