>>>> "Kevin" == Kevin Oberman
<oberman(a)es.net> writes:
Kevin> Here is the latest run. I had to compile frame.c with gcc (cc gave me
Kevin> a compiler interal error). Other than that, here is the result
Kevin> including all warnings I received.
Kevin> [...]
Kevin> cc -c -O2 -Demacs -I. -DHAVE_CONFIG_H -I/usr/local/include -D_BSD
-I/usr/dt/include search.c
Kevin> cc: Warning: search.c, line 749: In this statement, & before array
"(fastmap)" is ignored.
Kevin> xzero (fastmap);
Kevin> --^
Kevin> cc -c -O2 -Demacs -I. -DHAVE_CONFIG_H -I/usr/local/include -D_BSD
-I/usr/dt/include frame-x.c
Kevin> cc: Warning: frame-x.c, line 2585: # not in column 1 is ignored, skipping to end
of line.
Kevin> #ifdef HAVE_TOOLBARS
Kevin> --^
Kevin> cc: Warning: frame-x.c, line 2591: # not in column 1 is ignored, skipping to end
of line.
Kevin> #endif /* HAVE_TOOLBARS */
More trivial fixes.
1998-05-18 Martin Buchholz <martin(a)xemacs.org>
* frame-x.c (x_update_frame_external_traits): Start preprocessor
directives in column 1.
* search.c (skip_chars): Avoid using xzero with arrays, since some
compilers get confused by the construct &array.
--- src/frame-x.c.old
+++ src/frame-x.c
@@ -2582,13 +2582,13 @@
XtSetValues (FRAME_X_TEXT_WIDGET (frm), av, ac);
- #ifdef HAVE_TOOLBARS
+#ifdef HAVE_TOOLBARS
/* Setting the background clears the entire frame area
including the toolbar so we force an immediate redraw of
it. */
if (EQ (name, Qbackground))
MAYBE_DEVMETH (XDEVICE (frm->device), redraw_frame_toolbars, (frm));
- #endif /* HAVE_TOOLBARS */
+#endif /* HAVE_TOOLBARS */
/* Set window manager resize increment hints according to
the new character size */
--- src/search.c.old
+++ src/search.c
@@ -746,7 +746,7 @@
p = XSTRING_DATA (string);
pend = p + XSTRING_LENGTH (string);
- xzero (fastmap);
+ memset (fastmap, 0, sizeof (fastmap));
Fclear_range_table (Vskip_chars_range_table);