Not to throw shade, just wishing that somebody here can understand. Whenever an input is reasonably long, an analyzing function will crash, and this PR aims to fix that with a mechanism that contradicts the maintainer’s understanding while a similar C implementation does not need this fix. Clearly, the maintainer has not heard a certain programming mantra…

  • Giooschi@lemmy.world
    link
    fedilink
    English
    arrow-up
    7
    ·
    7 months ago

    They tested the same strings on that implementation

    The strings were the same, but not the implementation. They were testing the decoding of the strings, but the C function they were looking at was the one for encoding them. The decoding function was correct but what it read didn’t match the encoding one.

    though judging by the recent comments someone’s found something.

    Yeah, that’s me :)