dice icon indicating copy to clipboard operation
dice copied to clipboard

Handle INT64_MIN edge case in EncodeInt/DecodeInt

Open JyotinderSingh opened this issue 1 year ago • 2 comments

The current implementation of EncodeInt and DecodeInt may not correctly handle the minimum 64-bit integer value due to potential overflow in two's complement arithmetic. We need to add a specific test case for this value and ensure correct behavior. Tasks:

Requirements

  • Add a test case for INT64_MIN in int_test.go
  • Verify correct encoding and decoding of INT64_MIN
  • If issues are found, implement a fix in EncodeInt and/or DecodeInt functions

JyotinderSingh avatar Aug 03 '24 14:08 JyotinderSingh

Hey @JyotinderSingh I would like to pick this.

Vsareen0 avatar Aug 03 '24 15:08 Vsareen0

Hey @JyotinderSingh I would like to pick this.

Assigned.

JyotinderSingh avatar Aug 03 '24 15:08 JyotinderSingh