"Kristoffer Haugsbakk" <code@xxxxxxxxxxxxxxx> writes: > On Sat, Feb 10, 2024, at 10:48, Chris Torek wrote: >> On Sat, Feb 10, 2024 at 1:46 AM Kristoffer Haugsbakk >> <code@xxxxxxxxxxxxxxx> wrote: >>> I forgot tests. >> >> You presumably also wanted the `_` here for gettext-ing: >> >>> + die("%s: argument must be a non-negative integer", "padding"); >> >> Chris > > Yeah, thanks. You probably saved me a v3. :) > > Although I failed to notice that the string I stole was just a plain > string, not a translation string. And apparently there are no generic > “non-negative” translation strings. So I’ll just make a new one. The last time I took a look, I thought there were more than just the single entry point you patched that can feed negative padding into the machinery? Don't you need to cover them as well? Thanks.