From: Junio C Hamano <gitster@pobox.com>
To: Johannes Gilger <heipei@hackvalue.de>
Cc: Git Mailing List <git@vger.kernel.org>,
Johannes Schindelin <Johannes.Schindelin@gmx.de>
Subject: Re: [PATCHv2] git-log --format: Add %B tag with %B(x) option
Date: Fri, 18 Sep 2009 12:12:39 -0700 [thread overview]
Message-ID: <7vtyz083tk.fsf@alter.siamese.dyndns.org> (raw)
In-Reply-To: 1253296845-17219-1-git-send-email-heipei@hackvalue.de
Johannes Gilger <heipei@hackvalue.de> writes:
> %B(c5) is simply no ident, while %B(5c) is 5 spaces indent. Don't know if this
> is unwanted behaviour, but that's what strtol gives us.
Not really.
"strto[u]l(arg, &endp, 10)" parses arg as a decimal and moves "char *endp"
to point at where the number ended, so you can tell things like:
- return value of 0 with (arg == endp) being "an empty input, the user
did not necessarily meant zero"; or
- (*endp != '\0') being "some garbage after the number".
next prev parent reply other threads:[~2009-09-18 19:13 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-17 22:47 [PATCH] git-log --format: Add %B tag with %B(x) option Johannes Gilger
2009-09-17 23:27 ` Junio C Hamano
2009-09-18 18:00 ` [PATCHv2] " Johannes Gilger
2009-09-18 19:12 ` Junio C Hamano [this message]
2009-09-19 9:58 ` [PATCHv3] " Johannes Gilger
2009-09-22 19:41 ` Junio C Hamano
2009-09-22 21:30 ` [PATCHv4] git-log --format: Add %B tag with %B(n) option Johannes Gilger
2009-09-23 20:34 ` [PATCH 0/3] Add a pretty format to rewrapping/indenting commit messages Johannes Schindelin
2009-09-23 20:34 ` [PATCH 1/3] print_wrapped_text(): allow hard newlines Johannes Schindelin
2009-09-23 20:34 ` [PATCH 2/3] Add strbuf_add_wrapped_text() to utf8.[ch] Johannes Schindelin
2009-09-23 20:34 ` [PATCH 3/3] Add "%w" to pretty formats, which rewraps the commit message Johannes Schindelin
2009-09-23 21:00 ` Johannes Gilger
2009-09-23 23:19 ` Junio C Hamano
2009-10-05 6:25 ` Junio C Hamano
2009-09-24 0:00 ` [PATCH 1/3] print_wrapped_text(): allow hard newlines Linus Torvalds
2009-09-24 0:19 ` Johannes Schindelin
2009-09-25 8:21 ` Johannes Schindelin
2009-10-10 0:57 ` [PATCHv4] git-log --format: Add %B tag with %B(n) option Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7vtyz083tk.fsf@alter.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=git@vger.kernel.org \
--cc=heipei@hackvalue.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.