It's not just in CVS. It appears that it also made it from CVS into the
xemacs-21.2.37.tar.gz, since I ran into it last night when I downloaded
and compiled with MSVC 5.0:
ftp://ftp.xemacs.org/xemacs/beta/xemacs-21.2/xemacs-21.2.37.tar.gz
So might want to refresh that snapshot as well, or quickly turn
out a 21.2.38... not sure which is the preferred approach.
Thought you'd like to know though.
Troy
-----Original Message-----
From: Andy Piper [mailto:andyp@bea.com]
Sent: Tuesday, November 14, 2000 12:35 PM
To: James N. Potts
Cc: xemacs-beta(a)xemacs.org; xemacs-nt(a)xemacs.org
Subject: Re: Fw: Build failure, 21.2, current cvs, VC5
Oops, that wasn't mean't to get committed.
andy
At 12:43 PM 11/14/00 -0600, you wrote:
I meant to copy you on this.
-----Original Message-----
From: James N. Potts <jpmail(a)limolink.com>
To: xemacs-beta(a)xemacs.org <xemacs-beta(a)xemacs.org>
Date: Tuesday, November 14, 2000 12:40 PM
Subject: Build failure, 21.2, current cvs, VC5
When I try building from a fresh cvs checkout, I get:
]event-msw.c
]H:\src\XEMACS~1.2\nt\..\src\event-msw.c(2799) : error C2143:
] syntax error : missing ';' before '}'
]NMAKE : fatal error U1077: 'cl' : return code '0x2'
]Stop.
This comes from Andy's patch:
]1.38.2.61 Tue Nov 7 23:24:26 2000 by andyp
]CVS Tags: r21-2-37, r21-2-latest-beta; Branch: release-21-2
]Diffs to 1.38.2.60
]gutter fix
>--- xemacs/src/event-msw.c 2000/09/20 02:38:34 1.38.2.60
>+++ xemacs/src/event-msw.c 2000/11/07 23:24:26 1.38.2.61
[...]
>@@ -2792,6 +2794,8 @@
> case CBN_SELCHANGE:
> if (!NILP (mswindows_handle_gui_wm_command (frame, cid, id)))
> return 0;
>+ case BN_SETFOCUS:
>+
> }
> /* menubars always must come last since the hashtables do not
> always exist*/
Is there a reason for this empty case? In any case (pun not intended), I
just pulled out my copy of Harbison & Steele, and there must be a
statement after a case-label. A bare semi-colon does the trick.
-Jim Potts
--------------------------------------------------------------
Dr Andy Piper
Principal Consultant, BEA Systems Ltd