Funkwhale/Clementine-audio-player-Mac-Linux-Windows
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Add fallthrough comment where fallthrough is intended

Browse Source
This commit is contained in:
Jonas Kvinge 2020-01-04 23:29:19 +01:00
parent 0308a3f9a5
commit 39cdb7fc3f
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/globalsearch/globalsearchmodel.cpp
View File

@ -145,20 +145,25 @@ QStandardItem* GlobalSearchModel::BuildContainers(const Song& s,
case LibraryModel::GroupBy_Composer: case LibraryModel::GroupBy_Composer:
display_text = s.composer(); display_text = s.composer();
// fallthrough
case LibraryModel::GroupBy_Performer: case LibraryModel::GroupBy_Performer:
display_text = s.performer(); display_text = s.performer();
// fallthrough
case LibraryModel::GroupBy_Disc: case LibraryModel::GroupBy_Disc:
display_text = s.disc(); display_text = s.disc();
// fallthrough
case LibraryModel::GroupBy_Grouping: case LibraryModel::GroupBy_Grouping:
display_text = s.grouping(); display_text = s.grouping();
// fallthrough
case LibraryModel::GroupBy_Genre: case LibraryModel::GroupBy_Genre:
if (display_text.isNull()) display_text = s.genre(); if (display_text.isNull()) display_text = s.genre();
// fallthrough
case LibraryModel::GroupBy_Album: case LibraryModel::GroupBy_Album:
unique_tag = s.album_id(); unique_tag = s.album_id();
if (display_text.isNull()) { if (display_text.isNull()) {
display_text = s.album(); display_text = s.album();
} }
// fallthrough // fallthrough
case LibraryModel::GroupBy_AlbumArtist: case LibraryModel::GroupBy_AlbumArtist:
if (display_text.isNull()) display_text = s.effective_albumartist(); if (display_text.isNull()) display_text = s.effective_albumartist();
display_text = LibraryModel::TextOrUnknown(display_text); display_text = LibraryModel::TextOrUnknown(display_text);

5
src/library/librarymodel.cpp
View File

@ -1105,14 +1105,19 @@ LibraryItem* LibraryModel::ItemFromSong(GroupBy type, bool signal,
case GroupBy_Composer: case GroupBy_Composer:
item->key = s.composer(); item->key = s.composer();
// fallthrough
case GroupBy_Performer: case GroupBy_Performer:
item->key = s.performer(); item->key = s.performer();
// fallthrough
case GroupBy_Grouping: case GroupBy_Grouping:
item->key = s.grouping(); item->key = s.grouping();
// fallthrough
case GroupBy_Genre: case GroupBy_Genre:
if (item->key.isNull()) item->key = s.genre(); if (item->key.isNull()) item->key = s.genre();
// fallthrough
case GroupBy_Album: case GroupBy_Album:
if (item->key.isNull()) item->key = s.album(); if (item->key.isNull()) item->key = s.album();
// fallthrough
case GroupBy_AlbumArtist: case GroupBy_AlbumArtist:
if (item->key.isNull()) item->key = s.effective_albumartist(); if (item->key.isNull()) item->key = s.effective_albumartist();
item->display_text = TextOrUnknown(item->key); item->display_text = TextOrUnknown(item->key);