From: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
To: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [6683] Update cocoa.m to match new DisplayState code (Samuel Benson)
Date: Thu, 05 Mar 2009 11:05:58 +0000 [thread overview]
Message-ID: <49AFB216.6070300@eu.citrix.com> (raw)
In-Reply-To: <E1LewiZ-00077L-5I@cvs.savannah.gnu.org>
Anthony Liguori wrote:
> Revision: 6683
> http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=6683
> Author: aliguori
> Date: 2009-03-04 19:25:22 +0000 (Wed, 04 Mar 2009)
> Log Message:
> -----------
> Update cocoa.m to match new DisplayState code (Samuel Benson)
>
> Version 2 does as follows:
>
> [1]: Corrects endianness on issues by using native BGR to RGB conversion
> [2]: Uses DisplayState accessors for obtaining graphics context information,
> which
> [3]: Removes now unused variables, and
> [4]: Allows reading of varying color modes (32bit/24/16), and converting to
> native colorspace
> [5]: Attempts to keep itself centered on screen (as opposed to bottom right,
> which immediately goes off screen after bios load) on context changes
> (window resizes)
>
> Testing working on i386 (gentoo, Windows 2000) and PPC (debian) guests on PPC
> and x86 Macs.
>
> In regards to [4], Windows 2000 displays fine on quick tests, but on the lowest
> setting I could test, 16bit color depth at 4bpp, colors are slightly off. I
> used gentoo install-x86-minimal-2008.0 in framebuffer mode to test above
> setting; the usual grey text is now blue, and Tux appears to be BGR shifted. I
> do not know if previous code worked at such a low color setting.
>
> Signed-off-by: Samuel Benson <qemu_ml@digitalescape.info>
>
Please read this message:
http://lists.gnu.org/archive/html/qemu-devel/2009-03/msg00034.html
prev parent reply other threads:[~2009-03-05 11:07 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-04 19:25 [Qemu-devel] [6683] Update cocoa.m to match new DisplayState code (Samuel Benson) Anthony Liguori
2009-03-05 11:05 ` Stefano Stabellini [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=49AFB216.6070300@eu.citrix.com \
--to=stefano.stabellini@eu.citrix.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).