[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [bug-gawk] Increased RAM memory usage in gawk 5.0.1 compared to gawk
From: |
arnold |
Subject: |
Re: [bug-gawk] Increased RAM memory usage in gawk 5.0.1 compared to gawk 4.1.4 |
Date: |
Mon, 12 Aug 2019 08:12:00 -0600 |
User-agent: |
Heirloom mailx 12.5 7/5/10 |
"Andrew J. Schorr" <address@hidden> wrote:
> On Sun, Aug 11, 2019 at 07:42:37AM -0600, address@hidden wrote:
> > BTW, your change to disable the 4.2 IGNORECASE speed-up is fine for your
> > program, which doesn't use IGNORECASE, but is wrong for the general
> > case, so I cannot include it in the code base.
>
> Mightn't it be possible to stop populating re_reg[1] with the ignorecase
> variant when the node is created and instead populate it only when needed?
> As far as I can tell from a cursory inspection, the only place that
> re_reg[IGNORECASE]
> is accessed is from re.c:re_update. Could it make sense then to populate the
> regexps on a lazy basis from that function when they are needed? I guess
> this could be done for both the regular and IGNORECASE variants, but maybe
> the regexp needs to be compiled initially to give timely error messages...
>
> Regards,
> Andy
I have been thinking along the same lines. That would help with the
space issue.
I am right now trying to track down the reason that the execution
time is so much greater; I don't think it's related to IGNORECASE
change.
Arnold
Re: [bug-gawk] Increased RAM memory usage in gawk 5.0.1 compared to gawk 4.1.4, arnold, 2019/08/25