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

Re: [PATCH 3/3] pretty: support multiline subjects with format:

From: Markus Heidelberg <hidden>
Date: 2016-06-15 22:45:50

René Scharfe, 28.12.2008:
Markus Heidelberg schrieb:
quoted
quoted
For consistency and increased code reuse, change format: to do the
same as the other options.
What's wrong with using the first line instead of the first paragraph
for the other pretty options and for cases where only a small subject
line is desired? A sentence would be broken into a small uncomplete part
for the subject for example. What else?
I'm inching towards the goal of being able to replicate all of the other
--pretty flavours with format: (or tformat:) strings.

--pretty=oneline used to print only the first line, but was changed to
the way it's now in 4234a761 more than a year ago.  In my mind,
"subject" is defined by "what oneline is printing", so I think changing
the format: placeholder %s to show the whole first paragraph is a bug fix.
Yes, now I think so, too. I had also noticed the unexpectedly different
behaviour once.
If you actually like the old behaviour then perhaps a new placeholder
(%S ?) should be introduced instead?
I don't need it. I rather asked regarding the old behaviour for oneline
and so, but reading that there are strong intentions for their
behaviour, it's fine.

Markus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help