On Wed, May 20, 2015 at 12:27 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Johannes Schindelin <johannes.schindelin@xxxxxx> writes:
>
>>>> - fprintf(stderr,
>>>> - _("There are no candidates for merging among the refs that you just fetched.\n"
>>>> - "Generally this means that you provided a wildcard refspec which had no\n"
>>>> - "matches on the remote end.\n"));
>>>> + if (opt_rebase)
>>>> + fputs(_("There is no candidate for rebasing against among the refs that you just fetched."), stderr);
>>>
>> The puts() function appends a <newline> while fputs() does not.
>
> Yup, so this update makes the command spew unterminated lines, which
> not something intended...
Ugh >< Will put the "\n" back.
And yes, I used fputs() because it seems wasteful to use fprintf()
which will scan the string looking for any '%' formatting units, when
we know there aren't.
I will also update 05/14 to use fputs() as well where appropriate.
Thanks,
Paul
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html