D27105: Add const values 'nRows' and 'nCols'

Yanjie He noreply at phabricator.kde.org
Sun Feb 2 17:49:35 GMT 2020


yanjiehe added inline comments.

INLINE COMMENTS

> chehrlic wrote in commondefs.h:45
> static const -> constexpr

Thank you for your comment! But I doubt that change static const -> constexpr will have any improvement in performance or readability. Basic they are doing the same thing in this situation. I would still prefer using 'static const' because the code does not involve any computation in compile time.

REPOSITORY
  R411 KReversi

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

To: yanjiehe, #kde_games
Cc: chehrlic, kde-games-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20200202/ffe686b0/attachment.html>


More information about the kde-games-devel mailing list