On Tue, Aug 10, 2021 at 10:04 PM Marlon Richert wrote: > > On Sun, Aug 8, 2021 at 1:41 AM Bart Schaefer wrote: > > > > On Sat, Aug 7, 2021 at 12:56 PM Marlon Richert wrote: > > > > > > How does this patch look to you? > > > > Frankly, it looks unnecessarily complicated. What's the rationale for > > using COLUMNS=999 as a command prefix in every "eval", rather than > > just declaring once > > local -x COLUMNS=999 > > at the start of the function?? Why rearrange the other declarations? > > TIL you can export parameters locally. Here's a new patch. And since we're on the topic, this patch adds `-x` to `local` completion.