On Wed, Nov 23, 2022 at 02:33:50PM +0900, Yoichi Nakayama wrote: > On Wed, Nov 23, 2022 at 3:54 AM Jeff King <peff@xxxxxxxx> wrote: > > Hmm, I know I suggested using a temporary file since "cat $tmpfile" > > should be pretty safe. But it does still have problems if your tmp > > directory has spaces. Or even other metacharacters, which I think will > > be interpreted by the eval, since $@ is expanded in the outermost level > > of the shell. > > Right. But the problem is not specific to emacs (it happens in vim too). > Let's fix it another time (as you noted, that's pretty unlikely, and we may > not even need to fix it). Good point. The vim version is easier to fix (we just need to double-quote \$1 inside the eval), but the fact that nobody has complained is an indication that it does not really matter. -Peff
