Re: xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-16 Thread Marc Aurele La France
On Tue, 14 Mar 2006, Thomas Dickey wrote: On Tue, 14 Mar 2006, David Dawes wrote: and adds this one: main.c:2586: warning: `pty_search' defined but not used yes, that's an annoyance (but the proposed fix reversed the general process of removing special definitions from main.c). Please

Re: xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-16 Thread Thomas Dickey
On Thu, 16 Mar 2006, Marc Aurele La France wrote: On Tue, 14 Mar 2006, Thomas Dickey wrote: On Tue, 14 Mar 2006, David Dawes wrote: and adds this one: main.c:2586: warning: `pty_search' defined but not used yes, that's an annoyance (but the proposed fix reversed the general process of

Re: xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-16 Thread Marc Aurele La France
On Thu, 16 Mar 2006, Thomas Dickey wrote: On Thu, 16 Mar 2006, Marc Aurele La France wrote: On Tue, 14 Mar 2006, Thomas Dickey wrote: On Tue, 14 Mar 2006, David Dawes wrote: and adds this one: main.c:2586: warning: `pty_search' defined but not used yes, that's an annoyance (but the

Re: xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-16 Thread Thomas Dickey
On Thu, 16 Mar 2006, Marc Aurele La France wrote: main.c, rather than refining it in xterm.h as a fallback for a configure script test. That's irrelevent. My fix holds whether xterm is built through imake or no, it is relevant. I considered a change like that some time ago, decided that

Re: xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-16 Thread Marc Aurele La France
On Thu, 16 Mar 2006, Thomas Dickey wrote: On Thu, 16 Mar 2006, Marc Aurele La France wrote: main.c, rather than refining it in xterm.h as a fallback for a configure script test. That's irrelevent. My fix holds whether xterm is built through imake or no, it is relevant. I considered a

Re: xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-15 Thread Thomas Dickey
On Tue, 14 Mar 2006, David Dawes wrote: On Tue, Mar 14, 2006 at 06:49:49PM -0500, Thomas Dickey wrote: than those addressed by the fix for lastlog(). That was using the configure script of course. Perhaps having the particular -D's set by imake would help focus the discussion. gcc -c -O2

xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-14 Thread David Dawes
On Sun, Mar 12, 2006 at 05:28:02PM -0800, Thomas Dickey wrote: CVSROOT: /home/x-cvs Module name: xc Changes by:[EMAIL PROTECTED] 06/03/12 17:28:02 Log message: 241. Xterm patch #210 (Thomas Dickey). This reinstates the following warnings on Solaris 9: xterm.h:243: warning:

Re: xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-14 Thread Thomas Dickey
On Tue, 14 Mar 2006, David Dawes wrote: On Sun, Mar 12, 2006 at 05:28:02PM -0800, Thomas Dickey wrote: CVSROOT:/home/x-cvs Module name:xc Changes by: [EMAIL PROTECTED] 06/03/12 17:28:02 Log message: 241. Xterm patch #210 (Thomas Dickey). This reinstates the following

Re: xterm warning regressions (was Re: CVS Update: xc (branch: trunk))

2006-03-14 Thread David Dawes
On Tue, Mar 14, 2006 at 06:49:49PM -0500, Thomas Dickey wrote: On Tue, 14 Mar 2006, David Dawes wrote: On Sun, Mar 12, 2006 at 05:28:02PM -0800, Thomas Dickey wrote: CVSROOT: /home/x-cvs Module name: xc Changes by: [EMAIL PROTECTED] 06/03/12 17:28:02 Log message: 241. Xterm patch