Compiler warnings unnecessarily caused by X11 headers

corresponding changes in many other declarations). Doing this right will, however, also be a good check that these string are not modified by the X11 library code. Regards Peter Breitenlohner /* compile with: * gcc -O2 -Wall -Wwrite-strings -Wcast-qual -c play.c */ #include #include #inclu

Re: [ANNOUNCE] xf86-input-evdev 2.1.2

ing a manpage formatting problem (made quite some time ago, but I forgot to post it). Regards Peter Breitenlohner From a686d6be70f8811838afacf0738eb13a1ec268f6 Mon Sep 17 00:00:00 2001 From: Peter Breitenlohner Date: Fri, 21 Nov 2008 21:57:41 +0100 Subject: [PATCH] fix manpage formatting --- man/evde

Re: DESTDIR and pkg-config variables

s: If the string "$PKG_CONFIG --variable=prefix xt" is a prefix of the string "$PKG_CONFIG --variable=appdefaultdir xt", then (and only then) replace that prefix by '${prefix}'. This will then be compatible with distcheck! regards Peter Breitenlohn

Re: Your recent changes to xorg-macros

So please apply. Thanks Peter Breitenlohner <[EMAIL PROTECTED]> ___ xorg mailing list xorg@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/xorg

Your recent changes to xorg-macros

to ANSI C. Thus, please modify the test such that they are included for gcc >= 3.4. Regards Peter Breitenlohner <[EMAIL PROTECTED]> ___ xorg mailing list xorg@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/xorg

Ansification of X.Org code: A question how to proceed

r is used subsequently without any test for this condition. ==== with best regards, Peter Breitenlohner <[EMAIL PROTECTED]> ___ xorg mailing list xorg@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/xorg

Re: Ansification of X.Org code & other cleanup work

: patches 20208-20212 have been submitted by me. They correct an obvious build failure along the lines already applied to app/xedit, but nevertheless they need independent review and approval. Regards Peter Breitenlohner <[EMAIL PROTECTED]> _

Re: candidate patches for server-1.5-branch inclusion

a PCI resource at 0x0005/65536. Without that patch this is erroneously reported as 0x0005/0 on 32Bit LE systems (at least on x86), or as 0x/327680 on 32Bit BE systems. Quite confusing! Regards, Peter Breitenlohner <[EMAIL P

Re: Is there a maintainer for proto/trapproto, lib/libXTrap, app/xtrap, and Xorg XTRAP extension?

apply them. Would you? Is there a way to move a file between modules while preserving the original author (git blame) of that file? Regards, Peter Breitenlohner <[EMAIL PROTECTED]> ___ xorg mailing list xorg@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/xorg

Is there a maintainer for proto/trapproto, lib/libXTrap, app/xtrap, and Xorg XTRAP extension?

not), and make sure any old installed version of is removed 3.3.2. Modify the Xorg XTRAP extension not to use int_function and void_function (may be somewhat complicated) 3.3.3. Remove these declarations from xtrapddmi.h. Regards, Peter Breitenlohner <[EMAIL PROTEC

Re: [PATCH] VPATH build of viewres, xedit, xfd, xgc, and xgc fails

On Mon, 10 Nov 2008, Peter Breitenlohner wrote: > There is, however, a radically different idea. Originally the app default > file, e.g., Xmessage was distributed as Xmessage.ad for the sake of case > insensitive filesystems in order not to cause conflict with the executable > xmessa

Re: [PATCH] VPATH build of viewres, xedit, xfd, xgc, and xgc fails

On Sun, 9 Nov 2008, Peter Breitenlohner wrote: > There is actually a much better way to avoid race conditions: whenever there > are several app-default files, make them depend on a phony target that > ensures the existence of ./app-defaults. The phony target I proposed yesterday

Your last commit for beforelight

Hi Jeremy, your commit from yesterday for beforelight (Buildfix for case insensitive file systems) has removed ./Beforelight.ad but failed to add ./app-defaults/Beforelight.ad. Please fix that. Regards Peter Breitenlohner <[EMAIL PROTECTED]> __

Re: [PATCH] VPATH build of viewres, xedit, xfd, xgc, and xgc fails

On Sat, 8 Nov 2008, Dan Nicholson wrote: On Fri, Nov 7, 2008 at 12:49 AM, Peter Breitenlohner <[EMAIL PROTECTED]> wrote: Attached are five tiny patches. Could one of you you please apply them. In my original message I had overlooked xmh with the same problem, patch is attached. In

Re: [PATCH] how to set CFLAGS variable for compiling Xorg

On Thu, 6 Nov 2008, Peter Breitenlohner wrote: Anyway, attached is a new version which does just the opposite: appending xorgversion.m4 to xorg-macros.m4.in instead of installing it as a separate file. Attached are two patches, the one from yesterday and my proposal for CWARNFLAGS (created

Re: patches: VPATH build of viewres, xedit, xfd, xgc, and xgc fails

On Fri, 7 Nov 2008, Dan Nicholson wrote: > On Fri, Nov 7, 2008 at 12:49 AM, Peter Breitenlohner <[EMAIL PROTECTED]> > wrote: >> >> Attached are five tiny patches. Could one of you you please apply them. > > In order to guard against races with parallel jobs,

patches: VPATH build of viewres, xedit, xfd, xgc, and xgc fails

ne of you you please apply them. Regards, Peter Breitenlohner <[EMAIL PROTECTED]>From 48801cf45c64a0ca9ea1e35142ee975b0a33fdff Mon Sep 17 00:00:00 2001 From: Peter Breitenlohner <[EMAIL PROTECTED]> Date: Thu, 6 Nov 2008 21:02:13 +0100 Subject: [PATCH] enabled VPATH build --- Makefile.am

Re: how to set CFLAGS variable for compiling Xorg

On Wed, 5 Nov 2008, Peter Breitenlohner wrote: On Wed, 5 Nov 2008, Julien Cristau wrote: this is in a separate (new) file xorg-changelog-cmd.m4, such that we can use it before it is installed meh. we can just depend on xorg-macros 1.2... Hi Julien, yes (of course) for all other modules

Re: how to set CFLAGS variable for compiling Xorg

On Wed, 5 Nov 2008, Julien Cristau wrote: >> this is in a separate (new) file xorg-changelog-cmd.m4, >> such that we can use it before it is installed > > meh. we can just depend on xorg-macros 1.2... Hi Julien, yes (of course) for all other modules, but not for xorg-macros which is about to ins

Re: how to set CFLAGS variable for compiling Xorg

al or conditional (e.g., via --enable-extra-warnings); yet to be decided. I'm not so sure about -pedantic for gcc, that warns about 'long long', extra long strings, and others we can hardly avoid. Regards Peter Breitenlohner <[EMAIL PROTECTED]>From 597894712187f6c3941dcc

Re: how to set CFLAGS variable for compiling Xorg

EFUN([XORG_CHANGELOG_CMD], [ CHANGELOG_RULE='' AC_SUBST([CHANGELOG_CMD]) ]) # XORG_CHANGELOG_CMD in configure.ac: XORG_CHANGELOG_CMD and in the top-level Makefile.am: ChangeLog: @CHANGELOG_CMD@ === That way the det

Re: how to set CFLAGS variable for compiling Xorg

On Wed, 5 Nov 2008, Julien Cristau wrote: >> this very same problem occurs in many (most?) modules. If you'd apply them I >> could prepare patches for that (against current git). >> >> There are, however, two or three related problems that maybe should be >> addressed at the same time. >> >> (1) I

Re: how to set CFLAGS variable for compiling Xorg

hem to you, provided you'd agree to apply them. If so, please also let me know your opinion about the additional warnings (never, conditional, or unconditional). Cheers, Peter Breitenlohner <[EMAIL PROTECTED]> ___ xorg mailing list xorg@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/xorg

Re: Ansification of X.Org code & other cleanup work

hat another xfs process is already running. This never happens because StorePid always return either 0 or -1, and would be problematic because there may well be two daemons using different ports. Fact is, however, that when I try to start a second daemon on the same port, that process overwrites the

Re: Ansification of X.Org code & other cleanup work (fwd)

Yesterday I sent this to the list, but not being subsribed it was of course rejected. regards Peter Breitenlohner <[EMAIL PROTECTED]> -- Forwarded message -- Date: Tue, 21 Oct 2008 11:05:13 +0200 (CEST) From: Peter Breitenlohner <[EMAIL PROTECTED]> To: Alan Coopers