Thread (8 messages) flat view 8 messages, 5 authors, 2016-06-15

Re: [PATCH] merge: default to @{upstream}

From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:50:28

Hi again,

Felipe Contreras wrote:
[out of order for convenience]
Anyway, I don't think we can set argv[1] to NULL, because it's
possible that this is "char *argv[1]", so that would crash.
The original argc is at least 1 (to include "merge" as argv[0]) before
the args are shifted by parse_options_end.  But probably that is too
tricky to rely on.
Sure, I need to add documentation and tests. I should probably have
sent this as 'RFC'.
No problem.  Thanks for keeping the topic alive.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help