Krazy complains about using QMessageBox here.
Details
Details
Diff Detail
Diff Detail
- Repository
- R412 KShisen
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
at this point i'd say that krazy warning is a bit old fashioned, but sure, won't hurt either
Comment Actions
Hmm, that was exactly my thought when I saw these Krazy issues. I think to remember exactly the opposite suggestion from a few years ago where Krazy said, we should use the equivalent Qt classes. Am I remembering correctly?
Comment Actions
Depends, KUrl? Yes Kurl is deprecated and you should use QUrl
KMessageBox is not deprecated and provides some more functionality than QMessageBox so it's ok to use
Comment Actions
Yes, I re-read the KF5 porting notes and I guess the KFoo -> QFoo thingy was a misconception based on the deprecation of quite some the KDE 4 KFoo classes.
Thanks for your comments. :)