Mercurial > hg > xemacs-beta
comparison configure @ 5016:2ade80e8c640
enable more warnings and fix them
-------------------- ChangeLog entries follow: --------------------
ChangeLog addition:
2010-02-08 Ben Wing <ben@xemacs.org>
* configure:
* configure.ac (TAB):
Various warnings that used to be present had mistakenly gotten
turned off. Turn them back on.
lwlib/ChangeLog addition:
2010-02-08 Ben Wing <ben@xemacs.org>
* xt-wrappers.h:
* xt-wrappers.h (Xt_SET_VALUE):
* xt-wrappers.h (Xt_GET_VALUE):
Rename var to avoid shadowing problems.
src/ChangeLog addition:
2010-02-08 Ben Wing <ben@xemacs.org>
* alloc.c:
Add prototypes for debugging functions.
* alloc.c (compact_string_chars): Make static.
* console-x.c (x_initially_selected_for_input):
* console-x.h:
* console-x.h (X_ERROR_OCCURRED):
Delete x_has_keysym() prototype from console-x.c, move to console-x.h.
* eval.c (multiple_value_call):
Real bug: Fix shadowing local vars.
* event-unixoid.c (read_event_from_tty_or_stream_desc):
* event-unixoid.c (signal_fake_event):
* lread.c (check_if_suppressed):
* strftime.c (strftime):
Fix stupid global shadowing warnings.
* event-unixoid.c (signal_fake_event):
* event-unixoid.c (drain_signal_event_pipe):
Use Rawbyte, not char.
* frame.h: Remove old prototype.
* gc.c:
* gc.c (show_gc_cursor_and_message):
* gc.c (remove_gc_cursor_and_message):
* gc.c (gc_prepare):
* gc.c (gc_finish_mark):
* gc.c (gc_finalize):
* gc.c (gc_sweep):
* gc.c (gc_finish):
* gc.c (gc_suspend_mark_phase):
* gc.c (gc_resume_mark_phase):
* gc.c (gc_mark):
* gc.c (gc_resume_mark):
Make fns static.
* glyphs-eimage.c (gif_decode_error_string):
Fix non-prototype.
* lisp.h:
Hack around global shadowing warnings involving `index'.
* intl-win32.c (wcsncpy):
* number-gmp.c (bigfloat_to_string):
* objects-msw.c (mswindows_font_spec_matches_charset_stage_2):
* specifier.c (call_charset_predicate):
* specifier.c (DEFINE_SPECIFIER_TAG_FROB):
Declarations cannot follow statements in standard C.
* search.c (search_buffer): Fix local shadowing warnings.
author | Ben Wing <ben@xemacs.org> |
---|---|
date | Mon, 08 Feb 2010 21:28:57 -0600 |
parents | 97c45e3ad810 |
children | b5df3737028a 861f2601a38b |
comparison
equal
deleted
inserted
replaced
5015:d95c102a96d3 | 5016:2ade80e8c640 |
---|---|
9644 | 9644 |
9645 with_cflags_warning="$with_cflags_warning -Wsign-compare -Wno-char-subscripts" | 9645 with_cflags_warning="$with_cflags_warning -Wsign-compare -Wno-char-subscripts" |
9646 test "$__GCC" -ge 3 && with_cflags_warning="$with_cflags_warning -Wpacked" | 9646 test "$__GCC" -ge 3 && with_cflags_warning="$with_cflags_warning -Wpacked" |
9647 test "$have_glibc" != "yes" && \ | 9647 test "$have_glibc" != "yes" && \ |
9648 with_cflags_warning="$with_cflags_warning -Wpointer-arith" | 9648 with_cflags_warning="$with_cflags_warning -Wpointer-arith" |
9649 if test "$XEMACS_CC_GPP" = "yes"; then | 9649 with_cflags_warning_c="-Wshadow -Wmissing-declarations" |
9650 xe_cflags_warning="$with_cflags_warning -Weffc++" | 9650 with_cflags_warning_c="$with_cflags_warning_c -Wmissing-prototypes -Wstrict-prototypes" |
9651 elif test "$__GCC" -ge 3; then | 9651 with_cflags_warning_cxx="-Weffc++" |
9652 xe_cflags_warning="$with_cflags_warning -Wunused-parameter" | 9652 with_cflags_warning_c_xe="" |
9653 if test "$__GCC" -ge 3; then | |
9654 with_cflags_warning_c_xe="$with_cflags_warning_c_xe -Wunused-parameter" | |
9653 if test "$__GCC" -gt 3 || test "$__GCC_MINOR" -ge 4; then | 9655 if test "$__GCC" -gt 3 || test "$__GCC_MINOR" -ge 4; then |
9654 with_cflags_warning="$with_cflags_warning -Wdeclaration-after-statement" | 9656 with_cflags_warning_c="$with_cflags_warning_c -Wdeclaration-after-statement" |
9655 fi | 9657 fi |
9656 fi | 9658 fi |
9657 with_cflags_warning="$with_cflags_warning -Wshadow -Wmissing-declarations" | 9659 if test "$XEMACS_CC_GPP" = "yes"; then |
9658 with_cflags_warning="$with_cflags_warning -Wmissing-prototypes -Wstrict-prototypes" | 9660 xe_cflags_warning="$with_cflags_warning $with_cflags_warning_cxx" |
9659 elif test "$__ICC" = "yes"; then | 9661 else |
9660 with_cflags_warning="-Wall -w1 -we147" | 9662 xe_cflags_warning="$with_cflags_warning $with_cflags_warning_c $with_cflags_warning_c_xe" |
9663 fi | |
9664 with_cflags_warning="$with_cflags_warning $with_cflags_warning_c" | |
9665 elif test "$__ICC" = "yes"; then | |
9666 with_cflags_warning="-Wall -w1 -we147" | |
9661 fi | 9667 fi |
9662 fi | 9668 fi |
9663 test -z "$xe_cflags_warning" && xe_cflags_warning="$with_cflags_warning" | 9669 test -z "$xe_cflags_warning" && xe_cflags_warning="$with_cflags_warning" |
9664 { $as_echo "$as_me:${as_lineno-$LINENO}: result: ${xe_cflags_warning}" >&5 | 9670 { $as_echo "$as_me:${as_lineno-$LINENO}: result: ${xe_cflags_warning}" >&5 |
9665 $as_echo "${xe_cflags_warning}" >&6; } | 9671 $as_echo "${xe_cflags_warning}" >&6; } |