tomahawk-player / tomahawk

Tomahawk, the multi-source music player
http://tomahawk-player.org
GNU General Public License v3.0
2.95k stars 367 forks source link

Fix "control reaches end of non-void function" in DelegateConfigWrapper #640

Closed Vogtinator closed 4 years ago

a17r commented 5 years ago

ping, this seems like an easy fix (indendation is off but that seems fuzzy in that file already).

dschmidt commented 5 years ago

If you still care, I'd rather put it underneath the switch so the compiler can still warn about unhandled enum values ...

hugolm84 commented 4 years ago

@Vogtinator Hi! I know this is a bit late, but if you do not mind, please see comments and update the PR. I will merge.

Best

Vogtinator commented 4 years ago

@Vogtinator Hi! I know this is a bit late, but if you do not mind, please see comments and update the PR. I will merge.

Best

Ok, done. Thanks!

hugolm84 commented 4 years ago

Thanks! :D

dschmidt commented 4 years ago

Amazing, thanks for getting this done after all <3