qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Oliver Gerlich <olig9@gmx.de>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Patch] Release mouse at window edges
Date: Wed, 02 Mar 2005 10:53:55 +0100	[thread overview]
Message-ID: <42258D33.9080402@gmx.de> (raw)
In-Reply-To: <42254590.4070905@wasp.net.au>

Brad Campbell wrote:
> Fabrice Bellard wrote:
> 
>> The idea is interesting. Does it cause regressions in the normal mode 
>> ? If so, an option to enable it may be good. I think it is a problem 
>> that your patch is dependent on the cursor size.
>>
> 
> This is a good point. I have had to revert it for WinXP as every time 
> the cursor changes shape (and it does all the time as you move over 
> stuff in Office2003) it releases the grab.
> 
> In addition I have had all manner of painting and rendering issues with 
> the IDG display driver and have had to revert to the built in driver to 
> get some software to paint properly.
> 
> Nice idea, might need a little work still.
> 
> Brad

An extra switch would be good, indeed. Currently, grabbing/ungrabbing is 
disabled if no hw cursor is used, but it's not reliable enough (for 
example, Linux initializes hw cursor at framebuffer initialization, 
although no cursor is visible).

The problem with cursor size appears differently on different systems. 
On Win98, hw_cursor_x goes from 1 to 1023 (in 1024x768), while on Win2k, 
it goes from 0 to 1013... The patch circumvents this problem with an 
ugly hack, but I have no idea how to get around this cleanly.

Brad: your problems with Office might be related to another bug I 
discovered (Internet Explorer 5 "disables" hw cursor by moving the 
cursor to 2047/2047 which ungrabs the cursor as soon as it enters an IE 
window). I'll post an improved patch tonight, with this and other bugs 
fixed.

BTW. is there a "make realclean" (or similar) which also deletes 
qemu-doc.html, manpages, kqemu temp files etc, so that I can easily diff 
my patched snapshot to an unpatched one?

Oliver Gerlich

  reply	other threads:[~2005-03-02 10:29 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-02-26 15:09 [Qemu-devel] [Patch] Release mouse at window edges olig9
2005-02-27  4:37 ` Brad Campbell
2005-02-27  6:16   ` Brad Campbell
2005-03-01 21:28 ` Fabrice Bellard
2005-03-02  4:48   ` Brad Campbell
2005-03-02  9:53     ` Oliver Gerlich [this message]
2005-03-02 11:24       ` Hetz Ben Hamo
2005-03-02 20:06       ` olig9

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=42258D33.9080402@gmx.de \
    --to=olig9@gmx.de \
    --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).