Michael Blow has posted comments on this change. Change subject: More Coverity Fixes ......................................................................
Patch Set 1: Oops, I misread the JLS transposing dividend & divisor. I think the logic for nextInt(MAX_VALUE) is a little more complicated than needed- I think masking off the sign bit will be the most efficient; will resubmit. -- 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: Michael Blow <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: No
