D15716: Suspicious use of signed arithmetic leads to negative index crash

[Shlomi Fish]

shlomif added a comment.


  Perhaps use the RNG code from https://fc-solve.shlomifish.org/faq.html#what_are_ms_deals which can handle up to 2**33 deal indices. Furthermore, does this code have unit tests?

REPOSITORY
  R410 KPatience

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

To: marten, #kde_games, fabiank
Cc: fabiank, shlomif, kde-games-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20181010/ce058918/attachment.html>