[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2
From: |
arnold |
Subject: |
Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2 |
Date: |
Thu, 12 Oct 2017 12:31:05 -0600 |
User-agent: |
Heirloom mailx 12.4 7/29/08 |
Hi All.
Thanks KO & Eli for working this out.
Eli Zaretskii <address@hidden> wrote:
> Thanks. Let's see if there are objections, and if not, I will push it
> in a few days in your name.
Minor nit:
> + char sym[strlen(name) + 1 + 1]; /* 1 for '-', 1 for NUL */
Should be /* 1 for '_', 1 for NUL */
Please fix in both places.
Otherwise looks good to me.
I will work on the fork issue Andreas mentioned.
Thanks!
Arnold
- [bug-gawk] [PATCH 0/2] Fix plugin issues on OS/2, KO Myung-Hun, 2017/10/11
- [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, KO Myung-Hun, 2017/10/11
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, Eli Zaretskii, 2017/10/11
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, KO Myung-Hun, 2017/10/12
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, Eli Zaretskii, 2017/10/12
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, KO Myung-Hun, 2017/10/12
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, Andrew J. Schorr, 2017/10/12
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, Eli Zaretskii, 2017/10/12
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2,
arnold <=
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, KO Myung-Hun, 2017/10/12
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, Eli Zaretskii, 2017/10/21
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, arnold, 2017/10/21
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, KO Myung-Hun, 2017/10/22
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, arnold, 2017/10/23
- Re: [bug-gawk] [PATCH 2/2] Fix plugin load failure on OS/2, KO Myung-Hun, 2017/10/22
[bug-gawk] [PATCH 1/2] Fix dlsym() failures on OS/2 kLIBC, KO Myung-Hun, 2017/10/11