D9182: return nullptr -> return {} for QFlags

2017-12-05 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R287:7f2c44add4aa: Use brace-initializer instead of nullptr in returning 0-QFlags (authored by mkoller). REPOSITORY R287 KImageFormats CHANGES SINCE LAST UPDATE

D9182: return nullptr -> return {} for QFlags

2017-12-05 Thread Kevin Funk
kfunk accepted this revision. kfunk added a comment. This revision is now accepted and ready to land. For the record, see comment https://phabricator.kde.org/D3987#174960 REPOSITORY R287 KImageFormats REVISION DETAIL https://phabricator.kde.org/D9182 To: mkoller, kfunk, kossebau Cc:

D9182: return nullptr -> return {} for QFlags

2017-12-04 Thread Friedrich W . H . Kossebau
kossebau added a comment. Code change looks good to me. Think I missed that in my clean-up commits to @kfunk 's initial commit as I only compiled the library parts of KDE Frameworks, not the plugin modules. No maintainer of this module though, so just a +1 for now. Commit message

D9182: return nullptr -> return {} for QFlags

2017-12-04 Thread Martin Koller
mkoller created this revision. mkoller added reviewers: kfunk, kossebau. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Change the irritating nullptr return value to {} for the QImageIOPlugin::Capabilities QFlags