> Wouldn't it have been simpler (and more generic) to keep the
> original prototype for graph_log() and then just change:
>
> command << sanitize(options.join(" "))
>
> to:
>
> command << options.shelljoin
Interesting. `shelljoin` is new to me. The reason I had to change it
around is that we don't really want to shell escape our own (safe)
arguments, only the ref name. Doing that would break our use of
`pretty=format:"..."`.
Christian
--
--
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
---
You received this message because you are subscribed to the Google Groups
"Gitorious" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
