On 09/30/2015 02:11 AM, Ben Pfaff wrote: > On Tue, Sep 29, 2015 at 04:46:37PM -0700, Ben Pfaff wrote: >> On Wed, Sep 23, 2015 at 02:25:47PM -0700, Andy Zhou wrote: >>> On Fri, Sep 18, 2015 at 3:26 PM, Ben Pfaff <b...@nicira.com> wrote: >>>> These functions could only work with 32-bit integers because of their >>>> special cases for an argument of value 0. However, none of the existing >>>> users depended on this special case, and some of the users did try to use >>>> these functions with 64-bit integer arguments. Thus, this commit changes >>>> them to support 64-bit integer arguments and drops the special cases for >>>> zero. >>>> >>> I wonder what would be the down side of returning 64 for zeros? >> >> Probably just an extra branch. It's conceptually a little weird though. > > I decided to apply this to master on the basis of Kyle's review. > > Returning 64 would be marginally safer though, even if it's slightly > more expensive. That might be the right trade-off, I'm not sure. >
I'm not 100% sure either. Returning 64 is still unsafe if unchecked, and if checked, why not just insure a non-zero precondition. Anyway, thanks for the fix, Ben. -Kyle This correspondence, and any attachments or files transmitted with this correspondence, contains information which may be confidential and privileged and is intended solely for the use of the addressee. Unless you are the addressee or are authorized to receive messages for the addressee, you may not use, copy, disseminate, or disclose this correspondence or any information contained in this correspondence to any third party. If you have received this correspondence in error, please notify the sender immediately and delete this correspondence and any attachments or files transmitted with this correspondence from your system, and destroy any and all copies thereof, electronic or otherwise. Your cooperation and understanding are greatly appreciated. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
