Re: [SeaBIOS] [PATCH] vgabios: Support emulated text in gfx_read_char()

[Paolo Bonzini]

On 06/01/2015 20:59, Paolo Bonzini wrote:
> On 06/01/2015 16:49, Kevin O'Connor wrote:
> > > +    if (vga_emulate_text()) {
> > > +        // Read bottom right pixel of the cell to guess bg color
> > > +        op.y += cheight-1;
> > > +        handle_gfx_op(&op);
> > > +        op.y -= cheight-1;
> > > +        bgattr = op.pixels[7];
> > > +        fgattr = bgattr ^ 0x7;
> > > +        car = 1;
> > > +    }
> > > It's better to use the top right pixel.  The bottom right has a false
> > > positive for characters 23 and 95.

And a bunch of others (I checked the bottom line instead of just the
bottom right).  But the top right has no false positive for ASCII
characters.

Paolo