On Fri, Sep 17, 2010 at 11:52:56AM +0200, Karel Zak wrote: > On Wed, Sep 08, 2010 at 05:43:31PM -0700, Chris Frost wrote: > > This corrects the display of the ext4 line that starts with a single quote. > > Do you mean > > 'journal_checksum' internally. > > in the man page, right? I do. > > One could fix this error by moving the quoted string. > > Yes ;-) > > > Instead, this patch also tries to prevent this error from occuring in > > the future. It changes the no-break control character to a character which > > is less likely to be used at the start of a line in this text (the semicolon). > > Wouldn't be better to keep it simple and stupid? I mean almost nobody > has a clue about .c2, it's probably elegant way how resolve one > problem and introduce another complexity to already horrible groff > stuff. > > > If changing this control character seems like a better way to go > > than actively avoiding the problem, it might be good to take the > > same approach in all util-linux-ng man pages. > > I guess the problem with the no-break control character is very > unusual -- if yes, then it's better to fix the broken lines rather > than add ".c2" to the all man pages. Simple and stupid is good, I agree. The reason I suggested the .c2 approach first is that text editors generally do not know that starting a line with ' is special. So asking a text editor to reflow a paragraph might regress. I'll post a second patch immediately after this email which moves the quoted string. I'm happy with either change. -- Chris Frost http://www.frostnet.net/chris/ -- To unsubscribe from this list: send the line "unsubscribe util-linux-ng" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
