Ian Maxon has posted comments on this change. Change subject: More Coverity Fixes ......................................................................
Patch Set 1: Maybe the Math.abs(rand.nextInt()) could get replaced with rand.nextInt(Integer.MAX_VALUE) instead? That's what the internets suggest, anyway, to avoid getting a negative when the number is too small to change to a positive without overflowing. -- To view, visit https://asterix-gerrit.ics.uci.edu/661 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0f63f7fb75b1ac8ffc1fc1d70e66bb19b788fe8e Gerrit-PatchSet: 1 Gerrit-Project: hyracks Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: No
