<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On 8 June 2015 at 23:58, Myriam Schweingruber <span dir="ltr"><<a href="mailto:myriam@kde.org" target="_blank">myriam@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Aroonav,<br>
<span><br>
On Mon, Jun 8, 2015 at 7:59 PM, Aroonav Mishra <<a href="mailto:aroonav11@gmail.com" target="_blank">aroonav11@gmail.com</a>> wrote:<br>
> The second week of GSoC coding period has come to an end. I didn't want to<br>
> create another thread for my weekly updates hence I have emailed in this<br>
> thread itself.<br>
<br>
</span>That's OK, actually a good idea :)<br>
<span>><br>
> So this week some of the changes that I made are as follows:<br>
><br>
</span><span>> Though KDialog is in KDELibs4Support but the classes that previously<br>
> inherited from this have changed their base class. As the usage of<br>
> functions(ex setButtons of KDialog) from these inherited classes cause<br>
> errors now so I am currently porting away from KDialog. I have to review<br>
> (many)changes made by the porting script for which I have to understand<br>
> their API first. So I believe this is going to take some time.<br>
<br>
</span>By "away from KDialog" you mean porting to QDialog, right?<br></blockquote><div>Yes I meant that.<br>And I was referring to the class Amarok2ConfigDialog which inherits from KConfigDialog which inherits from KPageDialog. Previously KPageDialog inherited from KDialog but in KF5 now it inherits directly from QDialog and skips KDialog. So the usage of functions(like setButtons which were part of KDialog) which were used in Amarok2ConfigDialog now cause errors.<br><br></div><div>So for this I am porting from KDialog to QDialog completely.<br><br></div><div>I hope this clears up everything.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span>><br>
> I will be pushing the commits that concern the above changes soon.<br>
><br>
> Cheers!!<br>
><br>
> P.S Happy Birthday Mamarok!<br>
<br>
</span>Thank you :)<br>
<br>
And thanks for the update, very appreciated!<br></blockquote><div>You are welcome :) <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Regards, Myriam<br>
<span><font color="#888888">--<br>
Proud member of the Amarok and KDE Community<br>
Protect your freedom and join the Fellowship of FSFE:<br>
<a href="http://www.fsfe.org" target="_blank">http://www.fsfe.org</a><br>
Please don't send me proprietary file formats,<br>
use ISO standard ODF instead (ISO/IEC 26300)<br>
_______________________________________________<br>
Amarok-devel mailing list<br>
<a href="mailto:Amarok-devel@kde.org" target="_blank">Amarok-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/amarok-devel" target="_blank">https://mail.kde.org/mailman/listinfo/amarok-devel</a><br>
</font></span></blockquote></div><br></div></div>