From: Eric Sunshine <sunshine@sunshineco.com>
Date: Mon, 26 Aug 2019 00:42:22 GMT
Subject: Re: [PATCH 08/13] t4014: let sed open its own files
Message-ID: <CAPig+cTBzxbEOo2tBOeTiYM8njDuAVTdBzATtPUmA6uMni4L-Q@mail.gmail.com>
In-Reply-To: <7d9a24a97972ab8a8a1cfe1f2605630e154886be.1566635008.git.liu.denton@gmail.com>

On Sat, Aug 24, 2019 at 4:27 AM Denton Liu <liu.denton@gmail.com> wrote:
> In some cases, we were using a redirection operator to feed input into
> sed. However, since sed is capable of opening its own files and provides
> better error messages on IO failure, make sed open its own files instead
> of redirecting input into it.

I don't care strongly one way or the other, but the justification of
"better error message" is rather weak:

    % sed '/^$/d' <foo
    sh: foo: No such file or directory
    % sed '/^$/d' foo
    sed: foo: No such file or directory
    %

I doubt it's worth a re-roll, though.

> Signed-off-by: Denton Liu <liu.denton@gmail.com>