Re: [PATCH] column: disallow negative padding

From: Kristoffer Haugsbakk <code@khaugsbakk.name>
Date: Feb 11, 2024, 17:10 UTC
Message-ID: <9c00311d-e31c-428b-9c66-fef7ac8bfc76@app.fastmail.com>
In-Reply-To: <CAPx1GvdDvmBmvoktd7onB4mSzikKf4eWVWnrzrn8c8Y1RcRgsA@mail.gmail.com>

On Sat, Feb 10, 2024, at 10:48, Chris Torek wrote:

Show 9 quoted lines

> On Sat, Feb 10, 2024 at 1:46 AM Kristoffer Haugsbakk
> <code@khaugsbakk.name> 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.

Cheers

-- 
Kristoffer Haugsbakk