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.
Details
Details
- Reviewers
markey - Group Reviewers
Amarok - Commits
- R181:1b81541eb2de: Port to new signal/slot syntax
Diff Detail
Diff Detail
- Repository
- R181 Amarok
- Branch
- qt5/kf5
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
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.
Comment Actions
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.
Comment Actions
There are no changes to the plugin system in this revision. Please try this and see if that fixes your issue.