From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23607 invoked by alias); 14 Apr 2012 15:29:07 -0000 Received: (qmail 23592 invoked by uid 22791); 14 Apr 2012 15:29:04 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=BAYES_00 X-Spam-Check-By: sourceware.org Received: from bearnaise.andreas-knepper.de (HELO bearnaise.andreas-knepper.de) (213.239.192.233) by sourceware.org (qpsmtpd/0.43rc1) with SMTP; Sat, 14 Apr 2012 15:28:50 +0000 Received: (qmail 13061 invoked from network); 14 Apr 2012 17:28:48 +0200 Received: by simscan 1.3.1 ppid: 13056, pid: 13057, t: 0.0564s scanners: clamav: 0.97.3 /m:46/d:7312 Received: from unknown (HELO blueshift.home.mensmann.net) (joerg?ml@bitplanet.de@92.225.36.76) by bearnaise.andreas-knepper.de with SMTP; 14 Apr 2012 17:28:48 +0200 Received: from joerg by blueshift.home.mensmann.net with local (Exim 4.76) (envelope-from ) id 1SJ4u7-0001GB-6w for cygwin-xfree@cygwin.com; Sat, 14 Apr 2012 17:28:47 +0200 From: =?iso-8859-1?Q?J=F6rg_Mensmann?= To: cygwin-xfree@cygwin.com Subject: [PATCH] Fix command line arguments for multiple monitors User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux) X-Emacs-Acronym: Emacs May Allow Customised Screwups X-Haustiere: Eine Gruppe doofer Sondereinsatz-Borneogibbons Date: Sat, 14 Apr 2012 15:29:00 -0000 Message-ID: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" Mailing-List: contact cygwin-xfree-help@cygwin.com; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-xfree-owner@cygwin.com Reply-To: cygwin-xfree@cygwin.com Mail-Followup-To: cygwin-xfree@cygwin.com X-SW-Source: 2012-04/txt/msg00047.txt.bz2 --=-=-= Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: quoted-printable Content-length: 175 Hi, placing the X server on a specific monitor using something like=20 "-screen 0 @2" is broken since November. The attached patch restores the old behaviour. ciao J=F6rg --=-=-= Content-Type: text/x-patch Content-Disposition: inline; filename=xwin-multi-monitor-fix.patch Content-length: 1090 [PATCH] hw/xwin: Fix command line arguments for multiple monitors. Moving Xwin to a certain monitor using "-screen 0 @2" would fail, printing "ddxProcessArgument - screen - Querying monitors failed". This happened since commit 3ead1d810b0e157078db39712e02ea6dc85216d8, because EnumDisplayMonitor() returns FALSE if its callback function returns FALSE (which is not clearly documented), and QueryMonitor() would then also return FALSE. Moving back to the old behaviour, where the return value of EnumDisplayMonitors() is ignored. --- hw/xwin/winmonitors.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/hw/xwin/winmonitors.c b/hw/xwin/winmonitors.c index 26e20b7..7baa3a1 100644 --- a/hw/xwin/winmonitors.c +++ b/hw/xwin/winmonitors.c @@ -63,6 +63,7 @@ QueryMonitor(int index, struct GetMonitorInfoData *data) data->requestedMonitor = index; /* query information */ - return EnumDisplayMonitors(NULL, NULL, getMonitorInfo, (LPARAM) data); + EnumDisplayMonitors(NULL, NULL, getMonitorInfo, (LPARAM) data); + + return TRUE; } -- 1.7.9 --=-=-= Content-Type: text/plain; charset=us-ascii Content-length: 223 -- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Problem reports: http://cygwin.com/problems.html Documentation: http://x.cygwin.com/docs/ FAQ: http://x.cygwin.com/docs/faq/ --=-=-=--