[PATCH] leak in rl_filename_completion_function

2023-05-31 Thread Grisha Levit
If rl_filename_rewrite_hook returns a new string for a filename (which I guess only happens on macOS with bash), it is in most cases not free-d. run() { for ((i=0; i<=1; i++)); do ((i%1000)) || ps -o rss= $BASHPID compgen -f . >/dev/null done } $ (run) 2160 4576 6864 9040

Re: enable builtin bugs in bash 5.2

2023-05-31 Thread alex xmb ratchev
On Wed, May 31, 2023, 16:58 Robert Elz wrote: > Date:Wed, 31 May 2023 10:07:07 -0400 > From:Chet Ramey > Message-ID: > > | The documentation can always be changed to match a change in behavior. > > Of course, but you only do that wen the doc is either unclear, or

Re: enable builtin bugs in bash 5.2

2023-05-31 Thread Robert Elz
Date:Wed, 31 May 2023 10:07:07 -0400 From:Chet Ramey Message-ID: | The documentation can always be changed to match a change in behavior. Of course, but you only do that wen the doc is either unclear, or when the behaviour change is obviously to something better.

Re: enable builtin bugs in bash 5.2

2023-05-31 Thread alex xmb ratchev
On Wed, May 31, 2023, 16:07 Chet Ramey wrote: > On 5/30/23 4:46 PM, Robert Elz wrote: > > Date:Tue, 30 May 2023 15:28:57 -0400 > > From:Chet Ramey > > Message-ID: > > > >| H. That's not the only option. How about we load it if found > but mark > >|

Re: enable builtin bugs in bash 5.2

2023-05-31 Thread Chet Ramey
On 5/30/23 4:46 PM, Robert Elz wrote: Date:Tue, 30 May 2023 15:28:57 -0400 From:Chet Ramey Message-ID: | H. That's not the only option. How about we load it if found but mark | it as not enabled? It will still take `enable -d' to delete it. That