Coverity: Reverse INULL: CID:207

* src/media/media.c (media_stream_finalise): Return immediately if the stream is NULL

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Coverity: Reverse Negative: CID:209

* src/mule/file-coding.c (coding_system_from_mask): Make sure cat
is >=0 before dereferencing in the loop.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

COVERITY: Forward NULL: CID:40

* lib-src/ootags.c (analyse_regex): Return if regex_arg is NULL

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

COVERITY: Forward NULL CID:46

* src/print.c (write_string_to_stdio_stream): Add appropriate
diagnostics if no console nor stream provided...
* src/print.c (Fexternal_debugging_output): Explict NULL
initialization.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

COVERITY: SECURE_TEMP - CID:345

* lib-src/movemail.c (lock_dot): Force umask to a sensible default, ie remove all group and other permissions

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Another go at fixing 131

* merges:
  Summary: fix, exec elisp in right thread context, closes bug #131 Keywords: bug fix #131
  Summary: revert, bring asyneq handling of job events to original state Keywords: bug unfix #131
  Summary: fix, another bug 131 attempt, dispatch the Keywords: bug fix #131

Merge branch 'bug/131' into next

* bug/131:
  Summary: fix, exec elisp in right thread context, closes bug #131 Keywords: bug fix #131
  Summary: revert, bring asyneq handling of job events to original state Keywords: bug unfix #131
  Summary: fix, another bug 131 attempt, dispatch the Keywords: bug fix #131

Summary: fix, exec elisp in right thread context, closes bug #131
Keywords: bug fix #131

This changeset now most definitely fixes the sentinel issue of bug #131,
this time the elisp is even executed in the main thread.

* src/events/event-stream.c: move asyneq events (work started, ...) to
  execute_internal_event(), was in Fdispatch_event()

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Byte-compiler warning fixes

* lisp-warnfix:
  Fix a couple of byte-compiler warnings.

Fix a couple of byte-compiler warnings.

* lisp/package-get.el (pui-bootstrap): Call #'pui-list-packages
via #'declare-fboundp to silence byte-compiler.
Also remove fake version hack.

* lisp/ffi/ffi-wand.el (liquid-rescale): Call
#'Wand:liquid-rescale via #'declare-fboundp to silence
byte-compiler.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Summary: revert, bring asyneq handling of job events to original state
Keywords: bug unfix #131

This effectively reverts all the changes made to the asyneq handling of
job events, and also brings back bug #131.

* src/events/worker-asyneq.c: revert

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Nuke the bitrotten GTK

* gtk1-removal:
  GTK eradication -- plug the holes left from removing gtk files.
  GTK eradication -- file removal
  GTK eradication -- the build chain.

GTK eradication -- plug the holes left from removing gtk files.

This changeset fixes all the problems that were the by-product of removing
all the gtk related files from the repo.

* configure.ac: Remove spurious gtk comment

* etc/Makefile.am (dist_etc_DATA): Remove gtkrc.

* lib-src/gnuclient.c (main): No more gtk.

* lisp/cus-edit.el (custom-display): No more gtk.

* lisp/device.el (device-type): Remove spurious gtk comment.

* lisp/device.el (make-gtk-device): Removed.

* lisp/dragdrop.el (gtk-start-drag): Removed.

* lisp/dragdrop.el (gtk-start-drag-region): Removed.

* lisp/faces.el (face-equal): Don't check for gtk device type.

* lisp/faces.el (make-face-bold): Remove gtk case.

* lisp/faces.el (make-face-italic): Ditto.

* lisp/faces.el (make-face-bold-italic): Ditto.

* lisp/faces.el (make-face-unbold): Ditto.

* lisp/faces.el (make-face-unitalic): Ditto.

* lisp/faces.el (init-device-faces): Don't init gtk, it's not
there.

* lisp/faces.el (init-frame-faces): Ditto.

* lisp/faces.el (init-global-faces): Ditto.

* lisp/font-menu.el (font-menu-preferred-resolution): Remove gtk

* lisp/font-menu.el (font-menu-size-scaling): Ditto.

* lisp/font.el (font-window-system-mappings): Ditto.

* lisp/frame.el (make-frame-on-device): Remove spurious gtk doc string.

* lisp/gnuserv.el (gnuserv-main-frame-function): No more gtk.

* lisp/gnuserv.el (gnuserv-visible-frame-function): Ditto.

* lisp/gnuserv.el (make-x-device-with-gtk-fallback): Removed.

* lisp/gnuserv.el (gnuserv-edit-files): Remove gtk stuff.

* lisp/menubar-items.el (default-menubar): Remove sample.gtkrc

* lisp/minibuf.el (read-color-completion-table): No gtk.

* lisp/specifier.el: Ditto.

* src/emacs.c (console_type_create_select_gtk): Removed.

* src/emacs.c (main_1): No gtk.

* src/events/event-stream.c (init_event_stream): No gtk.

* src/events/events.c: Don't #include console-gtk.h

* src/events/events.c (event_equal): No gtk.

* src/events/events.c (event_hash): Ditto.

* src/events/events.c (format_event_object): Ditto.

* src/general-slots.h: Remove gtk.

* src/inline.c: Don't #include gtk headers.

* src/lisp.h: No gtk.

* src/lrecord.h: Remove gtk types.

* src/symsinit.h: Remove gtk.

* src/ui/console.c (select_console_1): No gtk.

* src/ui/console.h: Ditto.

* src/ui/console.h (CONSOLE_TYPESYM_WIN_P): No gtk.

* src/ui/device.c (Fmake_device): No gtk.

* src/ui/device.c (delete_device_internal): Ditto.

* src/ui/device.h (DEVICE_GTK_P): Removed.

* src/ui/device.h (CHECK_GTK_DEVICE): Removed.

* src/ui/device.h (CONCHECK_GTK_DEVICE): Removed.

* src/ui/device.h (DEVICE_MSWINDOWS_P): What the?  REEEEMOOOOOVED!!!

* src/ui/device.h (CHECK_MSWINDOWS_DEVICE): Gah!  Gone!

* src/ui/device.h (CONCHECK_MSWINDOWS_DEVICE): Sheesh! Outta here!

* src/ui/faces.c (complex_vars_of_faces): No gtk.

* src/ui/frame.h (FRAME_GTK_P): Removed.

* src/ui/frame.h (CHECK_GTK_FRAME): Removed.

* src/ui/frame.h (CONCHECK_GTK_FRAME): Removed.

* src/ui/gutter.c (specifier_vars_of_gutter): No gtk.

* src/ui/redisplay.c (init_redisplay): No gtk.

* src/ui/toolbar.c: No gtk.

* src/ui/window.c (specifier_vars_of_window): Ditto.

* tests/Makefile.am (gtk_tests): Removed.

* tests/Makefile.am (EXTRA_DIST): Removed gtk tests

* src/events/events.h: Don't #include gdk stuff.

* m4/sxe-aux.m4 (__SXE_MAKE_ADDER): Nuke gtk.

* src/sysfile.h: No gtk.

* src/ui/glyphs.c (pixmap_to_lisp_data): Remove gtk version.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Summary: fix, another bug 131 attempt, dispatch the
Keywords: bug fix #131

* src/events/worker-asyneq.c: dispatch directly

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

GTK eradication -- file removal

Signed-off-by: Steve Youngs <steve@sxemacs.org>

GTK eradication -- the build chain.

The GTK code in SXEmacs has been stagnating since 2000, 4 years before
SXEmacs even came into existance.  It was GTK version 1.  Nobody, nowhere
uses GTKv1 any more, so this stuff is going to /dev/null hell!

* src/ui/Makefile.am: Don't add `Gtk' to SUBDIRS

* src/Makefile.am: Don't add gtk to pdumplibs.

* lisp/Makefile.am (corelispels): Remove all the gtk/gnome elisp.

* lisp/dumped-lisp.el: Don't dump any gtk lisp.

* m4/sxe-summary.m4: No more gtk.

* m4/sxe-gui.m4 (SXE_CHECK_XT_DEPENDENCIES): Remove gtk stuff.

* m4/sxe-gui.m4 (SXE_CHECK_GTK_DEPENDENCIES): Removed.

* m4/sxe-gui.m4 (SXE_CHECK_UI_SUFFICIENCY): Don't subst gtk obj's

* configure.ac: Remove all gtk nonsense.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Really fix bug 131, for real this time

* crash-test:
  Fix the fix that fixes the fix to fix bug 131

Fix the fix that fixes the fix to fix bug 131

* src/events/worker-asyneq.c (eq_worker_eaten_myself): Don't call
asyneq_handle_event()

* src/events/worker-asyneq.c (eq_worker_work_started): Ditto.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Closes bug #131 from Sebastian

* merges:
  Summary: cosmetics, clean up event-stream code Keywords: code hygiene
  Summary: fix, actually dispatch events, fixes bug #131 Keywords: bug fix #131

Merge branch 'code/hygiene' into next

* code/hygiene:
  Summary: cosmetics, clean up event-stream code Keywords: code hygiene

Summary: cosmetics, clean up event-stream code
Keywords: code hygiene

This changeset brings event-stream.c to 21st century coding style and
our 80 cols rule.

* src/events/event-stream.c: big refactoring

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Merge branch 'bug/131' into next

* bug/131:
  Summary: fix, actually dispatch events, fixes bug #131 Keywords: bug fix #131

Summary: fix, actually dispatch events, fixes bug #131
Keywords: bug fix #131

This changeset makes sure that the communication between REPL event
queue and delegation event queue (the one for the worker threads)
actually work in both directions.  Events put on the REPL event queue
from within jobs that have been delegated are now announced to the event
queue handler and hence executed, which in particular was the reason the
sentinel wasn't invoked as described in bug #131.

* src/events/worker-asyneq.c: call asyneq_handle_event() after enqueuing

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Build clean up, etc. from Sebastian

* merges:
  Summary: cosmetics, conditionalise seemingly unused vars in movemail.c Keywords: refactor warn-fix
  Summary: cosmetics, refactor large parts of make-docfile.c Keywords: refactor warn-fix
  Summary: fix, -Winline -> only if maximum warning output Keywords: bldchn code hygiene warn-fix
  Summary: fix, unused warnings in gnuserv.c Keywords: bldchn code hygiene warn-fix
  Summary: fix, signedness and unused warnings in gnuserv et al. Keywords: bldchn code hygiene warn-fix
  Summary: fix, clean up signedness problem in gnuserv and friends Keywords: bldchn code hygiene warn-fix
  Summary: cosmetics, declare yow internal functions static Keywords: bldchn
  Summary: cosmetics, fix some signedness and shadow warnings in gnuclient Keywords: bldchn
  Summary: cosmetics, fix some warnings about signedness Keywords: bldchn
  Summary: chore, deactivate some warnings again (if not aggressive) Keywords: bldchn

Merge branch 'bld/warn-fix' into next

* bld/warn-fix:
  Summary: cosmetics, conditionalise seemingly unused vars in movemail.c Keywords: refactor warn-fix
  Summary: cosmetics, refactor large parts of make-docfile.c Keywords: refactor warn-fix

Summary: cosmetics, conditionalise seemingly unused vars in movemail.c
Keywords: refactor warn-fix

This changeset conditionalises on MAIL_USE_POP to fix warnings that
arise from writing to unused variables.

* lib-src/movemail.c: put MAIL_USE_POP guards around poppass var

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: cosmetics, refactor large parts of make-docfile.c
Keywords: refactor warn-fix

This changeset refactors large parts of make-docfile.c, i.e. kill unused
code and kludges, adhere to our 80 column policy and maintain our coding
style throughout the file.

* lib-src/make-docfile.c: refactor

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Fix if/else scope in yow.c from Rudi

* merges-rudi:
  fix yow.c, include assert.h (was missing since cb837c1c)
  fix yow.c, correct else scope (broken since cb837c1c)

Merge branch 'bld/warn-fix' into next

* bld/warn-fix:
  Summary: fix, -Winline -> only if maximum warning output Keywords: bldchn code hygiene warn-fix

Summary: fix, -Winline -> only if maximum warning output
Keywords: bldchn code hygiene warn-fix

This changeset fixes some inline related warnings by turning them off.

* m4/sxe-acx_pthread.m4: enable -Winline only with_maximum_warning_output

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Merge branch 'bld/warn-fix' into next

* bld/warn-fix:
  Summary: fix, unused warnings in gnuserv.c Keywords: bldchn code hygiene warn-fix
  Summary: fix, signedness and unused warnings in gnuserv et al. Keywords: bldchn code hygiene warn-fix
  Summary: fix, clean up signedness problem in gnuserv and friends Keywords: bldchn code hygiene warn-fix

Summary: fix, unused warnings in gnuserv.c
Keywords: bldchn code hygiene warn-fix

This changeset brings gnuserv.c to 21st century coding style and
fixes warnings along.

Note to myself: gnuserv is not v6-ready!  Several locations assume for
inet addresses to fit in an unsigned long which is so 1980.

* lib-src/gnuserv.c: fix warnings and ennicen syntax

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: fix, signedness and unused warnings in gnuserv et al.
Keywords: bldchn code hygiene warn-fix

This changeset brings some of the stuff in gnuclient.c to 21st century
coding style and fixes warnings along.

* lib-src/gnuclient.c: fix warnings and ennicen syntax

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>
Conflicts:

lib-src/gnuclient.c

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: fix, clean up signedness problem in gnuserv and friends
Keywords: bldchn code hygiene warn-fix

This changeset introduces SNPRINTF, a macro that also ascertains that
its result is >=0 and < than the size of the buffer printed into.

* lib-src/gnuclient.c: migrate to SNPRINTF

* lib-src/gnuserv.c: migrate to SNPRINTF

* lib-src/gnuserv.h: provide SNPRINTF macro

* lib-src/gnuslib.c: migrate to SNPRINTF

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>
Conflicts:

lib-src/gnuclient.c
lib-src/gnuserv.c

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Merge branch 'bldchn' into next

* bldchn:
  Summary: cosmetics, declare yow internal functions static Keywords: bldchn
  Summary: cosmetics, fix some signedness and shadow warnings in gnuclient Keywords: bldchn
  Summary: cosmetics, fix some warnings about signedness Keywords: bldchn
  Summary: chore, deactivate some warnings again (if not aggressive) Keywords: bldchn

Summary: cosmetics, declare yow internal functions static
Keywords: bldchn

* lib-src/yow.c: make local things static

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: cosmetics, fix some signedness and shadow warnings in gnuclient
Keywords: bldchn

* lib-src/gnuclient.c: fix signedness warnings
  fix shadow warnings
  adhere to our coding style

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>
Conflicts:

lib-src/gnuclient.c

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: cosmetics, fix some warnings about signedness
Keywords: bldchn

* lib-src/gnuslib.c: treat ints returned by snprintf() as size_t for
  assertions

* src/regex.c: strictly return values from the enum as declared

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>
Conflicts:

lib-src/gnuslib.c
lib-src/profile.c

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: chore, deactivate some warnings again (if not aggressive)
Keywords: bldchn

This changeset demotes some of the warning flags again to the special
--with-maximum-warning-output option.

* m4/sxe-compiler.m4: conditionalise on some warning switches again

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Close bug #133 from Sebastian

* merges:
  Summary: fix, bug 133, for real now Keywords: bug fix
  Summary: cosmetics, prescind from using inlines for trivial one-off stuff Keywords: bldchn code hygiene
  Summary: fix, final, use unsigned ints in modifier setup, fixes bug #133 Keywords: bug fix
  Summary: fix, tentative, use unsigned int:1 to avoid overflows, fixes bug #133 Keywords: bug fix

Merge branch 'bug/133' into for-steve

* bug/133:
  Summary: fix, bug 133, for real now Keywords: bug fix

Summary: fix, bug 133, for real now
Keywords: bug fix

This changeset fixes 133 by doing everything right.

* src/ui/X11/event-Xt.c: actually reset the old modifier to the bit it
  represents, this should fix 133 once and forever

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Merge remote-tracking branch 'rudi/fixes' into for-steve-from-rudi

* rudi/fixes:
  fix yow.c, include assert.h (was missing since cb837c1c)
  fix yow.c, correct else scope (broken since cb837c1c)

fix yow.c, include assert.h (was missing since cb837c1c)

fix yow.c, correct else scope (broken since cb837c1c)

Summary: cosmetics, prescind from using inlines for trivial one-off stuff
Keywords: bldchn code hygiene

* src/ui/X11/event-Xt.c: superfluous inlines are removed

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Merge branch 'bug/133' into for-steve

* bug/133:
  Summary: fix, final, use unsigned ints in modifier setup, fixes bug #133 Keywords: bug fix

Summary: fix, final, use unsigned ints in modifier setup, fixes bug #133
Keywords: bug fix

This fixes bug 133 finally.

* src/ui/X11/event-Xt.c: make the bit set a bit wider

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Merge branch 'bug/133' into for-steve

* bug/133:
  Summary: fix, tentative, use unsigned int:1 to avoid overflows, fixes bug #133 Keywords: bug fix

Summary: fix, tentative, use unsigned int:1 to avoid overflows, fixes bug #133
Keywords: bug fix

This fixes bug 133 tentatively, a problem with the bitset seems to spoil
the big de-nestification.  However, it appears the bits have to be wider
than I planned them to be, this is up to investigation, could be an
alignment issue.

* src/ui/X11/event-Xt.c: make the bit set a bit wider

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

from Sebastian

* merges:
  Summary: chore, kick ase extern inline Keywords: code hygiene build clang
  Summary: chore, don't check for extern inline support anymore Keywords: code hygiene build clang
  Summary: chore, crunch the number of extern_inline definitions Keywords: code hygiene build clang
  Summary: chore, refactor nested function construct in event-Xt.c Keywords: build clang
  Summary: chore, replace nested function construct by cpp equivalent Keywords: build clang

Conflicts:
src/ui/X11/event-Xt.c  -- trivial, resolved.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Merge branch 'bld/clang' into next

* bld/clang:
  Summary: chore, kick ase extern inline Keywords: code hygiene build clang
  Summary: chore, don't check for extern inline support anymore Keywords: code hygiene build clang
  Summary: chore, crunch the number of extern_inline definitions Keywords: code hygiene build clang
  Summary: chore, refactor nested function construct in event-Xt.c Keywords: build clang
  Summary: chore, replace nested function construct by cpp equivalent Keywords: build clang

Summary: chore, kick ase extern inline
Keywords: code hygiene build clang

This changeset reviews ase code that erroneously used extern inlines
directly, transform them to static inlines.

* modules/ase/ase-cartesian.h: extern inline -> static inline

* modules/ase/ase-heap.h: extern inline -> static inline

* modules/ase/ase-neighbourhood.h: extern inline -> static inline

* modules/ase/ase.h: extern inline -> static inline

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: chore, don't check for extern inline support anymore
Keywords: code hygiene build clang

This changeset drops the check for extern inline support (and compiler
allergies to it) altogether, resorting instead to static inlines.

* m4/sxe-compiler.m4: do not check for extern inline compiler allergies

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: chore, crunch the number of extern_inline definitions
Keywords: code hygiene build clang

This changeset drops transitional extern_inline definitions so only pure
C99 static inlines are used from now on.

* src/sxe-utils.h: drop conditional extern_inline definitions leaving
  only static inline as expansion

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Warning fixes from Nelson

* merges:
  Remove casts which were not needed and causing comparison sign mismatch
  Add prototype for emacs_tparam to avoid warning
  Change the pre-processor conditional to check for MPC_SET_UI_FR to avoid warning
  Add new Assert-Equal and Assert-Not-Equal macros to test-harness, which print the actual values in case of failure. Use instead of (Assert (equal)) and (Assert (not (equal)))

Remove casts which were not needed and causing comparison sign mismatch

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Add prototype for emacs_tparam to avoid warning

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Change the pre-processor conditional to check for MPC_SET_UI_FR to avoid warning

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Add new Assert-Equal and Assert-Not-Equal macros to test-harness, which print the actual values in case of failure. Use instead of (Assert (equal)) and (Assert (not (equal)))

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Warning fixes from Nelson

* merges:
  More warning suppressions
  More warning suppression
  Get rid of the slew of warning in tags code

More warning suppressions

* lib-src/gnuslib.c (connect_to_unix_server): Use size_t for sz

* src/ui/X11/event-Xt.c (x_reset_modifier_mapping): Add full
prototype to auto functions

* src/ui/glyphs-eimage.c (png_instantiate): use png_uint_32 in loop

* src/ui/lwlib/xlwmenu.c (parameterize_string): add new offset to
keep signedness in comparison

* src/ui/X11/glyphs-x.c (x_subwindow_query_geometry): add prototyp

* lib-src/gnuslib.c (connect_to_internet_server): Use size_t to
compare with sizeof.

* src/alloc.c (garbage_collect): ditto.
* src/buffer.c (Fgenerate_new_buffer_name): ditto.
* src/dumper.c (pdump_file_try): ditto.
* src/dumper.c (pdump_load): ditto.
* src/editfns.c (Fencode_time): ditto.
* src/editfns.c (Fencode_btime): ditto.
* src/editfns.c (Fcurrent_time_zone): ditto.
* src/extents.c (print_extent_1): ditto.
* src/mule/input-method-xlib.c (EmacsXtCvtStringToXIMStyles): ditto.
* src/mule/input-method-xlib.c (describe_Window): ditto.
* src/print.c (SXE_VSNPRINT_VA): ditto.
* src/print.c (write_hex_ptr): ditto.
* src/print.c (printing_major_badness): ditto.
* src/process-unix.c (allocate_pty_the_old_fashioned_way): ditto.

* src/process-unix.c (unix_open_network_stream): ditto.
* src/process-unix.c (unix_open_network_server_stream): ditto.
* src/process.c (make_process_internal): ditto.
* src/strftime.c (add_num_time_t): ditto.
* src/ui/TTY/objects-tty.c (Ffind_tty_color): ditto.
* src/ui/TTY/objects-tty.c (Ftty_registered_color_list): ditto.
* src/ui/TTY/redisplay-tty.c (term_get_fkeys_1): ditto.
* src/ui/X11/device-x.c (read_locale_specific_resources): ditto.
* src/ui/X11/device-x.c (x_init_device): ditto.
* src/ui/X11/device-x.c (signal_if_x_error): ditto.
* src/ui/X11/dialog-x.c (dbox_descriptor_to_widget_value): ditto.
* src/ui/X11/event-Xt.c (x_keysym_to_emacs_keysym): ditto.
* src/ui/X11/event-Xt.c (describe_event): ditto.
* src/ui/X11/frame-x.c (color_to_string): ditto.
* src/ui/X11/frame-x.c (x_set_initial_frame_size): ditto.
* src/ui/X11/frame-x.c (Fx_window_id): ditto.
* src/ui/X11/objects-x.c (truename_via_random_props): ditto.
* src/ui/X11/scrollbar-x.c (x_create_scrollbar_instance): ditto.
* src/ui/glyphs-eimage.c (tiff_error_func): ditto.
* src/ui/glyphs-eimage.c (tiff_warning_func): ditto.
* src/ui/lwlib/lwlib-Xaw.c (make_dialog): ditto.
* src/ui/lwlib/lwlib-Xaw.c (make_dialog): ditto.
* src/ui/lwlib/xlwgauge.c (GaugeExpose): ditto.
* src/ui/lwlib/xlwgauge.c (MaxLabel): ditto.
* src/ui/lwlib/xlwmenu.c (parameterize_string): ditto.
* src/ui/lwlib/xlwtabs.c (XawTabsSetTop): ditto.
* src/ui/redisplay.c (decode_mode_spec): ditto.
* src/ui/redisplay.c (decode_mode_spec): ditto.
* src/ui/redisplay.c (decode_mode_spec): ditto.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

More warning suppression

* lib-src/etags.c (etags_strrchr): restore the register qualifier

* lib-src/gnuclient.c (main): remove unneeded shadowed variable definition

* lib-src/gnuclient.c (tell_emacs_to_resume): use size_to compare to sizeof
* lib-src/gnuclient.c (filename_expand): ditto.
* lib-src/gnuclient.c (main): ditto.
* lib-src/gnuserv.c (unix_init): ditto.
* lib-src/ootags.c (main): ditto.
* lib-src/profile.c (get_time): ditto.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Get rid of the slew of warning in tags code

* lib-src/etags.c (main): added missing case labels.
* lib-src/etags.c (consider_token): ditto.
* lib-src/etags.c (C_entries): ditto.
* lib-src/etags.c (main): Made sure size was compared to sizeof as size_t .

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Buildchain fix from Nelson

* merges:
  Fix previous fix. Use X_CFLAGS, not X_FLAGS...
  Revert removal of X_FLAGS from libsxeuiTTY_a_CPPFLAGS. We really need to fix this. TTY should not depend on X11 even if X11 is being linked in...
  Revert removal of X_FLAGS from libsxecore_a_CPPFLAGS and libcruft_a_CPPFLAGS

Fix previous fix. Use X_CFLAGS, not X_FLAGS...

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Summary: chore, refactor nested function construct in event-Xt.c
Keywords: build clang

This changeset replaces the nested function thing used for modifier
mappings by ordinary function calls and a closure structure.

* src/ui/X11/event-Xt.c (x_reset_modifier_mapping): auto inline foo ->

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: chore, replace nested function construct by cpp equivalent
Keywords: build clang

This changeset replaces a nested function left-over, introduced to
provide closure (over the compile stack), by a more portable
pre-processor construct.

* src/regex.c (regex_compile): auto inline foo -> #define

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Revert removal of X_FLAGS from libsxeuiTTY_a_CPPFLAGS. We really need to fix this. TTY should not depend on X11 even if X11 is being linked in...

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Revert removal of X_FLAGS from libsxecore_a_CPPFLAGS and libcruft_a_CPPFLAGS

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Merge branch 'trivial'

* trivial:
  fix my conflict fix

fix my conflict fix

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Buildchain fixes and updates from Sebastian

* merges: (21 commits)
  Summary: fix to fix, do not blank EMACSPACKAGEPATH when finding ... Keywords: bldchn
  Summary: minor, also massage sxemacs calls in tests/ with batch env Keywords: bldchn
  Summary: minor, put XCCLDFLAGS in libtool built binaries and libs Keywords: build icc
  Summary: minor, provide XCCLDFLAGS to be passed on through libtool to the CC Keywords: build icc
  Summary: minor, tweak SXE_CHECK_LINKER_FLAGS Keywords: bldchn
  Summary: fix to fix, do not treat compiler flags as linker flags Keywords: bldchn
  Summary: minor, specifically check for all optimisation flags Keywords: bldchn
  Summary: fix, check for special diagnostic 10237 if CC=icc Keywords: bldchn
  Summary: fix, use all CFLAGS for linker flag check Keywords: bldchn
  Summary: minor, provide SXE_CHECK_COMPILER_XFLAG Keywords: bldchn
  Summary: cosmetics, add meself to sppm.texi Keywords: boasting
  Summary: minor, always pass on SXE_DYLD_PATH with pointers to the lwlibdir Keywords: bldchn
  Summary: minor, call the linker (or the compiler front-end) when ... Keywords: bldchn
  Summary: fix, carefully quote esd lib prepending Keywords: bldchn
  Summary: minor, rewrite linker checks to use the compiler front-end and XFLAG Keywords: bldchn
  Summary: cosmetics, add ar-lib wrapper script to git ignore file Keywords: bldchn
  Summary: fix to cosmetics, add AM_PROG_AR conditionally Keywords: bldchn
  Summary: cosmetics, add AM_PROG_AR like autoreconf suggests Keywords: bldchn
  Summary: minor, check certain compiler flags regardless of CC flavour Keywords: bldchn
  Summary: minor, provide C11 check for anonymous structs Keywords: bldchn
  ...

Conflicts:
src/Makefile.am  -- trivial conflict resolved

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Merge branch 'bldchn' into next

* bldchn:
  Summary: fix to fix, do not blank EMACSPACKAGEPATH when finding ... Keywords: bldchn
  Summary: minor, also massage sxemacs calls in tests/ with batch env Keywords: bldchn

Summary: fix to fix, do not blank EMACSPACKAGEPATH when finding ...
Keywords: bldchn

... load path shadows.

* src/Makefile.am (sxemacs.dmp): use BATCHENV in target so load paths
  contain the current user's package paths

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: minor, also massage sxemacs calls in tests/ with batch env
Keywords: bldchn

Again, due to a libtool inconsistency, this changeset adds the location
of the newly built lwlib libary to the environment before calling
sxemacs, this time in tests/ so make check will work.

* tests/Makefile.am: add batch environment to sxemacs binary calls

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Merge branch 'bld/icc' into next

* bld/icc:
  Summary: minor, put XCCLDFLAGS in libtool built binaries and libs Keywords: build icc
  Summary: minor, provide XCCLDFLAGS to be passed on through libtool to the CC Keywords: build icc

Summary: minor, put XCCLDFLAGS in libtool built binaries and libs
Keywords: build icc

This changeset adds XCCLDFLAGS to all libs and binaries touched by
libtool, in general whenever there's a foo_la_bar foo_la_LDFLAGS or
AM_LDFLAGS should contain $(XCCLDFLAGS).

* modules/ase/Makefile.am: add XCCLDFLAGS

* modules/cl/Makefile.am: add XCCLDFLAGS

* modules/dbus/Makefile.am: add XCCLDFLAGS

* modules/ldap/Makefile.am: add XCCLDFLAGS

* src/Makefile.am: add XCCLDFLAGS

* src/ui/lwlib/Makefile.am: add XCCLDFLAGS

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: minor, provide XCCLDFLAGS to be passed on through libtool to the CC
Keywords: build icc

* m4/sxe-compiler.m4: put -static-foo into XCCLDFLAGS and AC_SUBST them

* m4/sxe-libtool.m4: prepend XCCFLAG (-XCClinker) to XCCLDFLAGS entries

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

MacOS libpng detection fixes from Nelson

* merges:
  Fix detection of MacOS X native provided libpng
  Overcome differences in order of CFLAGS vs argument list. In the process, make the libpng used in Mac OS be the one detected, not a difference between compile and runtime.

Merge branch 'bldchn' into next

* bldchn:
  Summary: minor, tweak SXE_CHECK_LINKER_FLAGS Keywords: bldchn
  Summary: fix to fix, do not treat compiler flags as linker flags Keywords: bldchn
  Summary: minor, specifically check for all optimisation flags Keywords: bldchn
  Summary: fix, check for special diagnostic 10237 if CC=icc Keywords: bldchn
  Summary: fix, use all CFLAGS for linker flag check Keywords: bldchn
  Summary: minor, provide SXE_CHECK_COMPILER_XFLAG Keywords: bldchn

Summary: minor, tweak SXE_CHECK_LINKER_FLAGS
Keywords: bldchn

Just to be symmetric provide a 4 argument version for macro
SXE_CHECK_LINKER_FLAGS, where the 4th argument is a list of additional
flags to pass to the linker, in analogy to the 4-arg version of
SXE_CHECK_COMPILER_FLAGS.

* m4/sxe-linker.m4: provide 4 argument SXE_CHECK_LINKER_FLAGS for
  analogy reasons

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: fix to fix, do not treat compiler flags as linker flags
Keywords: bldchn

This changeset reverts some checks for falsely dubbed linker flags which
are in fact compiler flags, namely -static-intel and -static-libgcc.

Because these flags need to be checked against the fully expanded
CFLAGS, the CFLAGS are generated beforehand and the checks themselves go
to FEATFLAGS.

* m4/sxe-linker.m4: fix, remove checks for -static-intel and
  -static-libgcc

* m4/sxe-compiler.m4: check for -static-intel and -static-libgcc in the
  context of FEATFLAGS

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Fix detection of MacOS X native provided libpng

* m4/sxe-mm.m4 (_SXE_MM_CHECK_PNG_VERSION): Use the previously
detected PNG LDFLAGS, CFLAGS and LIBS when determining version.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Summary: minor, specifically check for all optimisation flags
Keywords: bldchn

* m4/sxe-compiler.m4: if --with-ridiculously-aggressive-optimisations is
  not given perform a moderate check for optimisation flags instead of
  just setting them

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: fix, check for special diagnostic 10237 if CC=icc
Keywords: bldchn

* m4/sxe-compiler.m4: if CC=icc check for special diagnostic 10237 to
  facilitate -static-intel

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: fix, use all CFLAGS for linker flag check
Keywords: bldchn

* m4/sxe-linker.m4: actuaclly use the CFLAGS agreed upon so far to check
  linker flags

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: minor, provide SXE_CHECK_COMPILER_XFLAG
Keywords: bldchn

This changeset provides a convenience macro, SXE_CHECK_COMPILER_XFLAG,
to test how to pass on linker flags through the C compiler.

* m4/sxe-compiler.m4: new, SXE_CHECK_COMPILER_XFLAG

* m4/sxe-linker.m4: require SXE_CHECK_COMPILER_XFLAG

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: cosmetics, add meself to sppm.texi
Keywords: boasting

* info/sppm.texi: add my git repo

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Merge branch 'bldchn' into next

* bldchn:
  Summary: minor, always pass on SXE_DYLD_PATH with pointers to the lwlibdir Keywords: bldchn
  Summary: minor, call the linker (or the compiler front-end) when ... Keywords: bldchn
  Summary: fix, carefully quote esd lib prepending Keywords: bldchn
  Summary: minor, rewrite linker checks to use the compiler front-end and XFLAG Keywords: bldchn
  Summary: cosmetics, add ar-lib wrapper script to git ignore file Keywords: bldchn
  Summary: fix to cosmetics, add AM_PROG_AR conditionally Keywords: bldchn
  Summary: cosmetics, add AM_PROG_AR like autoreconf suggests Keywords: bldchn
  Summary: minor, check certain compiler flags regardless of CC flavour Keywords: bldchn
  Summary: minor, provide C11 check for anonymous structs Keywords: bldchn
  Summary: fix, use our compiler flag check macros to check for pthread{,s} Keywords: bldchn

Summary: minor, always pass on SXE_DYLD_PATH with pointers to the lwlibdir
Keywords: bldchn

This changeset prepends calls to the undumped or dumped sxe binary with
BATCHENV which holds paths to the newly built lwlib libs.  This fixes
builds that switch from say the athena widget kit to the lucid kit, as
the installed lwlib library will certainly not provide some symbols
needed with the new widget kit.

* lisp/Makefile.am: prepend all calls to sxe with BATCHENV

* modules/Makefile.am: prepend all calls to sxe with BATCHENV

* src/Makefile.am: prepend all calls to sxe with BATCHENV

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: minor, call the linker (or the compiler front-end) when ...
Keywords: bldchn

... checking for LD_ZFLAGs.  This facilitates sparse and clang builds.

* m4/sxe-linker.m4 (SXE_CHECK_LD_ZFLAG): be a wrapper for
  SXE_CHECK_LINKER_FLAGS

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: fix, carefully quote esd lib prepending
Keywords: bldchn

* configure.ac: fix, both esd-config and MM_LIBS quoted properly

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: minor, rewrite linker checks to use the compiler front-end and XFLAG
Keywords: bldchn

* configure.ac (SXE_CHECK_LINKER_FLAGS -> SXE_CHECK_LDFLAGS): rename, in
  accordance to SXE_CHECK_CFLAGS
  use new SXE_CHECK_LINKER_FLAGS macro to check for nocombreloc

* sxe-compiler.m4 (SXE_FEATFLAGS): move linker specific stuff to
  SXE_CHECK_LDFLAGS in sxe-linker.m4

* sxe-linker.m4 (SXE_CHECK_LINKER_FLAGS): new, macro to check for flags
  passed down to the linker via the compiler front-end, just as in
  real-life
  (SXE_CHECK_LDFLAGS): also check for compiler+linker specifics like
  static-intel and static-libgcc

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: cosmetics, add ar-lib wrapper script to git ignore file
Keywords: bldchn

* .gitignore: add `ar-lib'

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: fix to cosmetics, add AM_PROG_AR conditionally
Keywords: bldchn

* configure.ac: add AM_PROG_AR conditionally

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Summary: cosmetics, add AM_PROG_AR like autoreconf suggests
Keywords: bldchn

* configure.ac: add AM_PROG_AR

Signed-off-by: Sebastian Freundt <freundt@ga-group.nl>

Overcome differences in order of CFLAGS vs argument list. In the process, make the libpng
used in Mac OS be the one detected, not a difference between compile and runtime.

* m4/sxe-summary.m4 (SXE_SUMMARY): Add output of c_switch_system,
ld_switch_general and ld_libs_system to the summary.

* src/Makefile.am (libsxecore_a_CPPFLAGS): Remove X_CFLAGS since
they are not (and should not be) needed.
* src/Makefile.am (libcruft_a_CPPFLAGS): Ditto.
* src/ui/TTY/Makefile.am (libsxeuiTTY_a_CPPFLAGS): Ditto.

* src/Makefile.am (AM_CPPFLAGS): Use c_switch_general instead of c_switch_system
* src/database/Makefile.am (AM_CPPFLAGS): Ditto.
* src/ent/Makefile.am (AM_CPPFLAGS): Ditto.
* src/events/Makefile.am (AM_CPPFLAGS): Ditto.
* src/media/Makefile.am (AM_CPPFLAGS): Ditto.
* src/media/Makefile.am (libmm_a_CPPFLAGS): Ditto.
* src/mem/Makefile.am (AM_CPPFLAGS): Ditto.
* src/mule/Makefile.am (AM_CPPFLAGS): Ditto.
* src/ui/Gtk/Makefile.am (AM_CPPFLAGS): Ditto.
* src/ui/Gtk/Makefile.am (libsxeuiGtk_a_CPPFLAGS): Ditto.
* src/ui/Makefile.am (AM_CPPFLAGS): Ditto.
* src/ui/Makefile.am (libsxeui_a_CPPFLAGS): Ditto.
* src/ui/TTY/Makefile.am (AM_CPPFLAGS): Ditto.
* src/ui/TTY/Makefile.am (libsxeuiTTY_a_CPPFLAGS): Ditto.
* src/ui/X11/Makefile.am (AM_CPPFLAGS): Ditto.
* src/ui/X11/Makefile.am (libsxeuiX11_a_CPPFLAGS): Ditto.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Summary: minor, check certain compiler flags regardless of CC flavour
Keywords: bldchn

This changeset, in preparation to upcoming clang support, checks for
certain compiler flags (warnings at the moment) regardless of flavour.
It seems many compilers actually support a lot more gcc flags than gcc
itself, so check for them without checking for the flavour first.

This might break Sun studio build but Sun isn't shining anymore, is it?

* m4/sxe-compiler.m4: check for warning flags irrespective of CC flavour

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>

Summary: minor, provide C11 check for anonymous structs
Keywords: bldchn

This changeset provides a check for anon structs, an on-and-off feature
of gcc, and mandatory for C11 support.

* m4/sxe-compiler.m4 (SXE_CHECK_ANON_STRUCTS): simply check if anon
  structs are possible, they ought to be when -std=gnu1x is selected but
  that's only possible in gcc 4.6.1 it seems.

Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>