On Mon, Oct 29, 2012 at 8:28 AM, Kevin Krammer <span dir="ltr"><<a href="mailto:krammer@kde.org" target="_blank">krammer@kde.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
</div>I think you can remove m_accessToken, m_path and m_queryItems from<br>
FacebookJob.<br>
Access token and path are already set on m_url and addQueryItem can be<br>
implemented to just call m_url.addQueryItem().<br>
<br>
Also, virtual void start() = 0 is already part of KJob's API, so not needed<br>
again.<br></blockquote><div><br></div><div>Good points, all fixed.</div><div><br></div><div>Cheers</div></div>-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div><br>