[Po4a-devel]Re: [Po4a-commits] po4a/lib/Locale/Po4a Man.pm,1.72,1.73
- First Post
- Replies
- Stats
-
Go to
- ----- 2024 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2012 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2011 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2010 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2009 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2008 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2007 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2006 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2005 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2004 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
Urgh. I seem to remember that we restricted this to macros to avoid
rewriting verbatim sections. We had a man page containing
>>>>>
Here is an example of how to use the line
continuation with cpp:
#define my_macro some text being too long to \
be on one line only, but it's \
not a problem since we can use \
line continuation like this
Thanks for your attention.
<<<<<<
With you change, there is a risk that this verbatim block gets rewriten even
if it really shouldn't.
That being said, all this is based on remembrance. If it doesn't increase
the number of pages for which we fail, I must be wrong.
Thanks for your time,
Mt.
On Sun, Jan 09, 2005 at 06:04:10PM +0000, Nicolas FRAN??OIS wrote:
Update of /cvsroot/po4a/po4a/lib/Locale/Po4a
In directory haydn:/tmp/cvs-serv2440/lib/Locale/Po4a
Modified Files:
Man.pm
Log Message:
input line continuations (\<RET>) are not reserved for macros
Index: Man.pm
===================================================================
RCS file: /cvsroot/po4a/po4a/lib/Locale/Po4a/Man.pm,v
retrieving revision 1.72
retrieving revision 1.73
diff -u -d -r1.72 -r1.73
--- Man.pm 7 Jan 2005 22:50:52 -0000 1.72
+++ Man.pm 9 Jan 2005 18:04:07 -0000 1.73
@@ -290,7 +290,7 @@
$line =~ s/\\\././g;
chomp $line;
- while ($line =~ /^\..*\\$/ || $line =~ /^(\.[BI])\s*$/) {
+ while ($line =~ /^.*\\$/ || $line =~ /^(\.[BI])\s*$/) {
my ($l2,$r2)=$self->SUPER::shiftline();
chomp($l2);
if ($line =~ /^(\.[BI])\s*$/) {
_______________________________________________
Po4a-commits mailing list
Po4a-commits(a)lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/po4a-commits
Attachments:
- signature.asc (application/pgp-signature — 189 bytes)
Thanks Martin.
I will investigate this. The patch passed the regression test, but this
issue may have been hidden by wdiff.
My current answer is that in order to have a \ displayed in the man page,
a \\, \e or \E has to be used. \\ is changed to \e previously in
shiftline, and thus a line ending by a \ indicates an input line
continuation.
I need to test this with specific man pages.
I will also try to pass the regression test with diff instead of wdiff with and
without this patch.
Note: This patch only corrected 2 pages.
--
Nekral
On Tue, Jan 11, 2005 at 08:40:41PM +0100, Nicolas François wrote:
Thanks Martin.
I will investigate this. The patch passed the regression test, but this
issue may have been hidden by wdiff.
My current answer is that in order to have a \ displayed in the man page,
a \\, \e or \E has to be used. \\ is changed to \e previously in
shiftline, and thus a line ending by a \ indicates an input line
continuation.
I need to test this with specific man pages.
I still need to do this (and add these specific test to the man regression
test)
I will also try to pass the regression test with diff instead of
wdiff with
and without this patch.
This test passed without any diff (except dates of temporary files). I'm
really confident in this patch for the above reason (but consider a unit
test is still needed for latter regression tests).
--
Nekral
7250
days inactive
7285
days old
2 comments
2 participants
tags (0)
participants (2)
-
Martin Quinson -
Nicolas François