hpereiradacosta added inline comments.
INLINE COMMENTS
> breezeshadowhelper.cpp:177
> + QWidget* widget( static_cast<QWidget*>( object ) );
> + if( event->type() == QEvent::Paint && (
> !_widgets.contains(widget) || _widgets.value(widget) == 0 ) )
> + {
Also, instead of using first "contains" then .value, one should really use an
iterator (to avoid parsing the map twice):
auto&& iter = _widgets.find( widget );
if( iter == _widgets.end() || iter.value() == 0 )
Something like that.
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D5910
To: mart, #plasma, hpereiradacosta
Cc: anthonyfieroni, davidedmundson, plasma-devel, ZrenBot, spstarr, progwolff,
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
