Add two Q_ASSERT to detect incorrect protocols like:
- "" (empty parameter)
- smb:/// (the right one is smb)
- zip:/home/user/Download/......./Encoder.php (the right one is zip)
dfaure |
Frameworks |
Add two Q_ASSERT to detect incorrect protocols like:
Lint Skipped |
Unit Tests Skipped |
It's called protocol, it's supposed to be just the protocol. The bug is in the caller.
I've detected so far only the case of KProtocolInfo::icon with "", in a dependent revision.
Yes, but please don't push this until you have fixed the caller in your testcase, and used a Plasma/KF5 session with these asserts enabled for 2 weeks.
Thanks ;)