Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] for-each-ref: avoid loading objects to print %(objectname)

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:59:08

Jeff King [off-list ref] writes:
On Sat, Oct 26, 2013 at 10:35:17AM +0200, Thomas Rast wrote:
...
quoted
Can you quote the *?  I may have become somewhat paranoid, but still.
This is the first use of the *<field> syntax, and test_atom seems
written to correctly quote its arguments, so why risk it? :-)
Yeah, that is a very reasonable suggestion. And the patch is still in
pu, so it's not too late to squash.

Junio, here is a resend with the asterisks quoted, to replace what's in
jk/for-each-ref-skip-parsing.  I also double-checked that test_atom
keeps them properly quoted (it does).
Thanks, both, for being careful.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help