<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 10, 2014 at 12:43 AM, Albert Astals Cid <span dir="ltr"><<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
</span>I'm missing some documentation in the .h files you install from lib.<br></blockquote><div><br></div><div>All .h are now fully documented.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also it seems<br>
private Q_SLOTS:<br>
    void getCredentials();<br>
could go into the d pointer via the Q_PRIVATE_SLOT magic?<br></blockquote><div><br></div><div>Done.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
><br>
> KAccounts-providers is only a data repository, it contains the various XML<br>
> files needed for some basic accounts like Google and Facebook.<br>
><br>
> Signon-kwallet-extension is a plugin for signond (the upstream daemon) to<br>
> store secretes in kwallet.<br>
<br>
</span>Maybe the construct/destruct qDebugs you can turn into qcDebugs? Or plain<br>
remove them?<br></blockquote><div><br></div><div>Yeah, not too useful. Removed.</div></div><div><br></div><div>Cheers</div>-- <br><div class="gmail_signature"><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div></div>
</div></div>