Skip Menu |

This queue is for tickets about the HTML-FromANSI CPAN distribution.

Report information
The Basics
Id: 16436
Status: resolved
Priority: 0/
Queue: HTML-FromANSI

People
Owner: NUFFIN [...] cpan.org
Requestors: jpierce [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.01
Fixed in: (no value)



Subject: does not properly handle reverse
32 <ESC[4m for ESC[24mESC[7mChristESC[27mESC[1mmas should have Christ in reverse, but does not.
RT-Send-CC: AJWOOD [...] cpan.org
On Sun Dec 11 23:58:58 2005, JPIERCE wrote: Show quoted text
> 32 <ESC[4m for ESC[24mESC[7mChristESC[27mESC[1mmas > > should have Christ in reverse, but does not.
This is *probably* a Term::VT102 bug... The array-of-arrays representing the screen is grabbed from there, and the ordering is as found in the nested arrays. Hence, for this to work (without evil weirdness) Term::VT102 should return those character cells in reverse order, but i'm not 100% sure on this.
Subject: Re: [rt.cpan.org #16436] does not properly handle reverse
Date: Mon, 30 Jan 2006 08:31:34 +0000
To: ???????? ??????'???? via RT <bug-HTML-FromANSI [...] rt.cpan.org>
From: Andrew Wood <andrew.wood [...] ivarch.com>
On Sun, Jan 29, 2006 at 04:38:13PM -0500, ???????? ??????'???? via RT wrote: Show quoted text
> On Sun Dec 11 23:58:58 2005, JPIERCE wrote:
> > 32 <ESC[4m for ESC[24mESC[7mChristESC[27mESC[1mmas > > > > should have Christ in reverse, but does not.
> > This is *probably* a Term::VT102 bug... > > The array-of-arrays representing the screen is grabbed from there, and the ordering is as found > in the nested arrays. > > Hence, for this to work (without evil weirdness) Term::VT102 should return those character cells > in reverse order, but i'm not 100% sure on this.
Reverse video (ESC[7m) means that the background and foreground colours are reversed. It does not change the character ordering. - Andrew
CC: undisclosed-recipients: ;
Subject: Re: [rt.cpan.org #16436] does not properly handle reverse
Date: Mon, 30 Jan 2006 10:57:06 +0200
To: Andrew Wood via RT <bug-HTML-FromANSI [...] rt.cpan.org>
From: Yuval Kogman <nothingmuch [...] woobling.org>
On Mon, Jan 30, 2006 at 03:32:25 -0500, Andrew Wood via RT wrote: Show quoted text
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=16436 > > > On Sun, Jan 29, 2006 at 04:38:13PM -0500, ???????? ??????'???? via RT wrote:
> > On Sun Dec 11 23:58:58 2005, JPIERCE wrote:
> > > 32 <ESC[4m for ESC[24mESC[7mChristESC[27mESC[1mmas > > > > > > should have Christ in reverse, but does not.
... Show quoted text
> Reverse video (ESC[7m) means that the background and foreground colours are > reversed. It does not change the character ordering.
JPIERCE - Please tell me if the colors are reversed. If they are i'll close the bug. If not i'll reproduce, test, and finally get around to rereleasing a long awaited refactoring. Ciao! -- () Yuval Kogman <nothingmuch@woobling.org> 0xEBD27418 perl hacker & /\ kung foo master: /me supports the ASCII Ribbon Campaign: neeyah!!!
Download (untitled)
application/pgp-signature 189b

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #16436] does not properly handle reverse
Date: Mon, 30 Jan 2006 09:30:42 -0500
To: bug-HTML-FromANSI [...] rt.cpan.org
From: Jerrad Pierce <belg4mit [...] MIT.EDU>
They are not, hence the bug report. See http://web.mit.edu/belg4mit/www/12/ for an example of what the output should be when executing http://web.mit.edu/belg4mit/www/12/mod12.pl -- H4sICNoBwDoAA3NpZwA9jbsNwDAIRHumuC4NklvXTOD0KSJEnwU8fHz4Q8M9i3sGzkS7BBrm OkCTwsycb4S3DloZuMIYeXpLFqw5LaMhXC2ymhreVXNWMw9YGuAYdfmAbwomoPSyFJuFn2x8 Opr8bBBidccAAAA= -- MOTD on Setting Orange, the 30th of Chaos, in the YOLD 3172: Any damn fool can making sweeping generalizations about damn fools.
CC: undisclosed-recipients: ;
Subject: Re: [rt.cpan.org #16436] does not properly handle reverse
Date: Mon, 30 Jan 2006 19:55:19 +0200
To: Jerrad Pierce via RT <bug-HTML-FromANSI [...] rt.cpan.org>
From: Yuval Kogman <nothingmuch [...] woobling.org>
On Mon, Jan 30, 2006 at 09:31:49 -0500, Jerrad Pierce via RT wrote: Show quoted text
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=16436 > > > They are not, hence the bug report.
Oh, so I just misunderstood the original report... Sorry =) It'll take me a few days to get around to this since a refactoring is due, and I have few tuits. Ciao! -- () Yuval Kogman <nothingmuch@woobling.org> 0xEBD27418 perl hacker & /\ kung foo master: /me climbs a brick wall with his fingers: neeyah!
Download (untitled)
application/pgp-signature 189b

Message body not shown because it is not plain text.

CC: undisclosed-recipients: ;
Subject: Re: [rt.cpan.org #16436] does not properly handle reverse
Date: Mon, 30 Jan 2006 19:55:19 +0200
To: Jerrad Pierce via RT <bug-HTML-FromANSI [...] rt.cpan.org>
From: Yuval Kogman <nothingmuch [...] woobling.org>
On Mon, Jan 30, 2006 at 09:31:49 -0500, Jerrad Pierce via RT wrote: Show quoted text
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=16436 > > > They are not, hence the bug report.
Oh, so I just misunderstood the original report... Sorry =) It'll take me a few days to get around to this since a refactoring is due, and I have few tuits. Ciao! -- () Yuval Kogman <nothingmuch@woobling.org> 0xEBD27418 perl hacker & /\ kung foo master: /me climbs a brick wall with his fingers: neeyah!
Download (untitled)
application/pgp-signature 189b

Message body not shown because it is not plain text.

fixed in 2.02... very silly bug. ( $fg, $bg ) = ( $bg, $fg ) if $rv was in the code, but later the variables were unused ;-)