<div dir="ltr"><div class="" id="post-body-7460304479980061438" itemprop="description articleBody">
<div dir="ltr" style="text-align:left">
Hello,<br>I would have sent this yesterday but I was in the middle of a commit which I wanted to include here. So sorry for the delay.<br>I had to take the third week off but it didn't affect my project much as I had started working before 25 May.<br>
Some of the changes that I have made over the last few days are as follows:<br>
<ul style="text-align:left"><li>I have added KF5::KCMUtils and KF5NewStuff components.</li><li>Most of the KDialog code has been moved to QDialog which included
changes like replacement of setButtons with QDialogButtonBox. Although
KDialog is deprecated and this conversion seems unnecessary at this
stage of porting, I encountered dependencies on classes in KF5 that now
inherits from QDialog instead of KDialog. Due to this some KDialog code
in Amarok had to be ported now rather than later. In my last commit (id 3102d83) I have removed some extra instances of QWidget which I had made in the previous commit (id 343a203<em></em>) and which don't seem necessary while I was porting away from setButtons in KDialog.<br></li><li>I made a few classes in Amarok to inherit from KPageDialog instead
of KDialog to let us use the buttonBox() function in the former which is
pretty useful.</li><li>I removed the definition of slotButtonClicked() slot in
deviceconfiguredialog.cpp as there were not any signals connecting to
it. The rest of the slotButtonClicked() calls were replaced with either
QDialog::accept() or QDialog::reject().</li><li>I discovered a function QDir::toNativeSeparators which is really
useful. On moving from KUrl to QUrl, I had to replace the calls to
KUrl::addPath(const QString& txt) with u.setPath(u.path() + '/' +
txt) [Copied straight from the docs ;)]. Now you can see that we have a
'/' in setPath which may or may not be the native separator of the
platform. So here the static function QDir::toNativeSeparators comes
into use and it will make the separators appropriate (if needed) for the
underlying operating system.<br></li><li>On moving from KIcon to QIcon, it is suggested in the docs that QIcon::fromTheme be used but when the parameter is a QPixMap object, the constructor of QIcon seems more appropriate.</li><li>I have removed an overloaded constructor in TrackLoader.cpp which is no longer required due to the current usage of QUrl.</li><li>Some of the minor changes include the fixing of the header files, correcting conversions between QUrl and QString, replacement of KUrl::AdjustPathOption with QUrl equivalents, <br></li></ul>
Please comment if any of the above changes made by me seem erroneous.<br><br>Cheers<br></div><div style="text-align:left">Aroonav<br></div><div dir="ltr" style="text-align:left">
<div>
</div>
</div>
</div>
<span class=""></span></div><div class="gmail_extra"><br><div class="gmail_quote">On 9 June 2015 at 00:22, Aroonav Mishra <span dir="ltr"><<a href="mailto:aroonav11@gmail.com" target="_blank">aroonav11@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"><br><div class="gmail_extra"><div class="gmail_quote"><span class="">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></span><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><span class=""><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></span><div>You are welcome :) <br></div><span class=""><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></span></div><br></div></div>
</blockquote></div><br></div>