graesslin added inline comments.

INLINE COMMENTS

> davidedmundson wrote in egl_context_attribute_builder.cpp:50
> Do you have some sort of checklist of C++11 features to use?
> 
> For a single integer, emplace_back is going to do the exact same thing as the 
> standard push_back.

If one always uses emplace one doesn't forget to use it when it matters

REPOSITORY
  R108 KWin

BRANCH
  context-attribute-builder

REVISION DETAIL
  https://phabricator.kde.org/D6396

To: graesslin, #kwin, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

Reply via email to