Re: sdiff UI enhancement

[Paul Eggert]

Chuck Swiger <address@hidden> writes:

> At least with diff-2.7 or thereabouts, invoking the
> external editor via "e l" or "e r" would always fail and return the
> warning from this line of diff code:
>
>    fatal ("Subsidiary editor failed");
>
> ...it appeared that perhaps the wstatus variable wasn't getting set
> from calling waitpid()?

You can double-chec, but I think that bug was fixed here:

2004-04-12  Paul Eggert  <address@hidden>

        * NEWS, configure.ac (AC_INIT): Version 2.8.6.
        ...
        * src/sdiff.c (check_child_status): Renamed from ck_editor_status, and
        accept a new arg MAX_OK_STATUS.  All callers changed.
        Handle status 126/127 as per POSIX.