Since Qt5Script is deprecated and virtually useless without bindings support I got rid of it.
I moved the lyrics functionality to LyricsManager but the rest I deleted without reimplementing it in C++.
Details
- Reviewers
- None
- Group Reviewers
Amarok
Diff Detail
- Repository
- R181 Amarok
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage
See, the problem with removing features like this is, there will be a huge outcry from users. That's guaranteed, even without knowing for sure how much the scripting feature is really used in the wild.
I was discussing about this with Heiko. Maybe QtScript could be replaced with QJSEngine, similar to how the following patch for another project attempts to do it?
I see your point about removing features. But I've taken a look at the most popular scripts on opendesktop.org and they all rely on Qt bindings. Unless there is some way to restore bindings support this feature is broken and people will complain anyway.
About replacing QtScript with QJSEngine: I think it's quite possible to do so, but I have my doubts if the development and maintenance work is worth it given our limited manpower.
Anyway, I'll open a new revision just with the patch that implements the lyrics functionality in C++.