< Amarok | Development
Amarok/Development/TODO/Refactoring
QUrls
- Use QUrl and store QUrl instead of always changing them to strings
- Often we see parts of code like below, it makes no sense. Just take the string if the URL is not used!!!
QUrl url = QUrl::fromLocalFile(dataPath( "data/playlists/test.m3u" )); QFile file (url.toLocalFile);
QDir
- Do not use QDir::toNativeSeparor or QDir::Separator when we don't want to show them to the user: Qt does everything with '/' (even on windows!!)
Self coherence
- In playlists, url.uidUrl() returns a QUrl, in tracks it returns url.url(). Idem for SqlPodcastChannel, ....
This page was last edited on 9 November 2015, at 16:56. Content is available under Creative Commons License SA 4.0 unless otherwise noted.