D16556: Modernize object declarations and initializations

Daniel Vrátil noreply at phabricator.kde.org
Thu Nov 1 14:19:12 GMT 2018


dvratil added inline comments.

INLINE COMMENTS

> dkurz wrote in entitytreemodeltest.cpp:179
> My intent was to replace `()` with uniform initialization whereever I touch code. If I got you right, you'd prefer the opposite, using parens where possible, so I would now change back all those braces. Feel free to stop me if I misunderstood to spare me some time.

If you are calling a non-default constructor, you should use `()`, if you are constructing an object that only has a default ctor and you are initializing members directly, then use `{}`.

REPOSITORY
  R165 Akonadi

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

To: dkurz, #kde_pim, dvratil
Cc: dvratil, kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20181101/59fab9ef/attachment.html>


More information about the kde-pim mailing list