Port to new signal/slot syntax
ClosedPublic

Authored by malteveerman on Oct 7 2017, 11:18 AM.

Details

Summary

This is the first part of my rather large pull request here: Reviewboard
It ports much of the code to the new signal/slot syntax, which makes it easier to spot errors during the rest of the porting process.
I changed some signal/slot names, because the new syntax doesn't handle overloads particularly well and added some new ones, because the new syntax doesn't handle default arguments at all.
Nothing too major, though.
This is the first time I created a revision with arcanist. I'm not sure it knows this is based on the kf5 branch. Let me know if I did anything wrong here.

Diff Detail

Repository
R181 Amarok
Branch
qt5/kf5
Lint
Lint Skipped
Unit
Unit Tests Skipped
malteveerman created this revision.Oct 7 2017, 11:18 AM
malteveerman edited the summary of this revision. (Show Details)
markey added a subscriber: markey.Oct 8 2017, 1:20 PM
markey accepted this revision.Oct 8 2017, 4:00 PM

Hi Malte, I've applied your patch with Arcanist and then pushed it to remote. Before pushing I had to fix your author name and email though. KDE git hook refused your original author name because it was just one word.

Please fix your settings so that your real name and email address appear as author.

This revision is now accepted and ready to land.Oct 8 2017, 4:00 PM
markey closed this revision.Oct 8 2017, 4:11 PM

Not sure if it's any help...

I have compiled the branch kf5 which seem to contain this patches.

Everything seems to go smoothly until the compilation finishes and I try to run the application:

"Amarok could not find any plugins. This indicates an installation problem."

And then it quits.

There are no changes to the plugin system in this revision. Please try this and see if that fixes your issue.