chore, remove TEXI2DVI redefinitions

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

chore, mention build chain .el files in lisp's Makefile.am

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

chore, mention headers needed for the ase-set emod

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

chore, move systty.h to src/ui/TTY/

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

chore, put sxe-utils.h into distribution

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

chore, remove VPATH fiddling from sxe-srctree.m4

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

chore, don't mention non-existing files in Makefile.am's

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

chore, name ase-resclass-test.el correctly

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

chore, don't mention non-existing files in Makefile.am's

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

Fix return on non-void function from Rudi

* merges:
  fix, return with no value, in function returning non-void

fix, return with no value, in function returning non-void

followup 4da01e95

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

Coverity fixes from Nelson

* merges:
  Refactor
  Fix build
  Refactor
  Refactoring
  reflow
  Coverity CID 400832 Out of bounds access
  Small optimization

Merge branch 'njsf-cov' into for-steve

Refactor

* src/mule/file-coding.c (autodetect_real_coding_system): New
function to reduce indentention of scope. Also as soon as stream
read fails, it returns, thus solving one of the coverity issues.
Several scope reduction refactorings.

* src/mule/file-coding.c (determine_real_coding_system): Call the
new function.

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

Fix build

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

Refactor

* src/mule/file-coding.c (determine_real_coding_system): Refactor
condition in inner for loop so that negative branch of the
condition will continue loop, and fall through is the old positive
condition branch.

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

Refactoring

* src/mule/file-coding.c (determine_real_coding_system): Static
local variable for the mime name chars and some added explicit
scope braces.

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

reflow

* src/mule/file-coding.c (determine_real_coding_system): Reflow to make it easier to read and refactor.

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

Coverity CID 400832 Out of bounds access

* src/editfns.c (Ftemp_directory): Use proper size argument with xstrncat

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

Small optimization

* src/emodules-ng.c (__emodng_open_append_exts): New remain variable to keep
invariant computation...
* src/emodules-ng.c (__emodng_open_prepend_paths_append_exts): Ditto.

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

Close Bug #135, fix test suite. From Sebastian

* merges:
  Summary: fix, initialise cat morphism array properly, harden the tests Keywords: fix bug 135
  fix, change order of bitset in lrecord struct, hash_cache last
  cosmetics, turn morphism extern_inlines into static inlines
  test, run test suite with MALLOC_PERTURB_ set to be extra-harsh to the code
  test, introduce regression test for one MALLOC_PERTURB_ triggered crash
  Summary: fix, make sure ase-heap emits all symbols it needs Keywords: bld/chn

Don't put the dumpfile in bin dir from Rudi

* merges:
  bldchn fix, don't install buildir's dumpfile
  bldchn, install dumpfile into archlibdir
  try to read dumpfile also from libarchdir
  why the hell we don't have always EXEC_PATH available
  bldchn cleanup, remove/re-use some unused dumpfile related AM variables

Merge branch 'bug/135' into next

* bug/135:
  Summary: fix, initialise cat morphism array properly, harden the tests Keywords: fix bug 135
  fix, change order of bitset in lrecord struct, hash_cache last
  cosmetics, turn morphism extern_inlines into static inlines
  test, run test suite with MALLOC_PERTURB_ set to be extra-harsh to the code
  test, introduce regression test for one MALLOC_PERTURB_ triggered crash

Summary: fix, initialise cat morphism array properly, harden the tests
Keywords: fix bug 135

This changeset fixes an issue of using uninitialised memory (bug 135).

* src/lrecord.h (set_lheader_implementation): properly initialise lheaders

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

fix, change order of bitset in lrecord struct, hash_cache last

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

cosmetics, turn morphism extern_inlines into static inlines

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

test, run test suite with MALLOC_PERTURB_ set to be extra-harsh to the code

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

test, introduce regression test for one MALLOC_PERTURB_ triggered crash

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

Merge branch 'bld/x32' into next

* bld/x32:
  Summary: fix, make sure ase-heap emits all symbols it needs Keywords: bld/chn

Summary: fix, make sure ase-heap emits all symbols it needs
Keywords: bld/chn

This changeset fixes an issue with the ase-heap emodule.  Ctors and
wrappers are now declared and defined in the file that they're used.

* modules/ase/ase-heap.h: kick static inlines, no need to expose them

* modules/ase/ase-heap.c: soak up static inlines decls from the header

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

bldchn fix, don't install buildir's dumpfile

This problem became visible since we create the symlink within
archlibdir instead of bindir.
Also we delete $(bindir)/sxemacs.dmp from older installations
because unfortunately sxemacs would still try this first.

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

bldchn, install dumpfile into archlibdir

Also we only install the one which is named with dump_id.

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

try to read dumpfile also from libarchdir

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

why the hell we don't have always EXEC_PATH available

Does this break anything?

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

bldchn cleanup, remove/re-use some unused dumpfile related AM variables

This does not change anything.

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

Coverity and build fixes from Nelson

* merges:
  Coverity: CID 400010: Improper use of negative value
  Coverity: CID 400001: Logically dead code
  Coverity: CID 400005: Division or modulo by zero
  Coverity: CID 400007: Deference after null check
  Coverity: CID 400831: Out of bonds access
  Coverity: CID 400832: Out of bounds access
  Fix build
  Fix the relocate_fd
  Coverity CID 400833 UNINIT

Coverity: CID 400010: Improper use of negative value

* src/ui/minibuf.c (echo_area_append): Make sure length<0 also
returns early here.

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

Merge branch 'for-steve' into njsf-cov

Coverity: CID 400001: Logically dead code

* lib-src/etags.c (get_language_from_filename): Rework code a
little so that Coverity does not get fooled by ternary operator.

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

Coverity: CID 400005: Division or modulo by zero

* src/media/sound-oss.c (sound_oss_play): Fix gross condition
error of previous change

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

Coverity: CID 400007: Deference after null check

* src/eval.c (Finteractive_p): Make sure btp is not NULL before
dereference.

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

Coverity: CID 400831: Out of bonds access

* src/fileio.c: Provide proper length to xstrncat

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

Coverity: CID 400832: Out of bounds access

* src/editfns.c (Ftemp_directory): Proper size passed to xstrncat

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

Fix build

* src/callproc.c (relocate_fd): fix the argument rename usage

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

Merge branch 'master' into for-steve

Fix the relocate_fd

* src/callproc.c (max_filedesc): New function to determine the
actual maximum file descriptor.

* src/callproc.c (relocate_fd): Realized previous use of just dup
was actually better, made sure that the closing of temporary duped
file descriptors was commented.

* src/emacs.c (__sys_stk_sz): Minor cosmetic variable rename.

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

Coverity CID 400833 UNINIT

* src/sysdep.c (raw_open): Use the proper argument for open

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

test suite fixes from Nelson

* merges:
  Why are quick fixes so bad ? Because they break more than they fix

Why are quick fixes so bad ? Because they break more than they fix

* src/emodules-ng.c (__emodng_open_prepend_paths): Fix xstpncpy
arguments. For real this time.

* src/emodules-ng.c (__emodng_open_prepend_paths_append_exts): Ditto.

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

test suite fixes from Nelson

* merges:
  Improve relocate_fd with dup2
  Fix build

Merge branch 'njsf-cov' into for-steve

Improve relocate_fd with dup2

* src/callproc.c (relocate_fd): Satisfy checkers for return values
until proper check of getrlimit is implemented

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

Fix build

* src/emodules-ng.c (__emodng_open_prepend_paths): Correct usage of xstpcpy size argument.
* src/emodules-ng.c (_adapt_load_path): Ditto.
* src/emodules-ng.c (__emodng_open_prepend_paths_append_exts): Fix indentation

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

Fix autoload breakage from prev changeset from Nelson

* merges:
  Fix introduced bug by coverity fix attempt

Fix introduced bug by coverity fix attempt

* src/search.c (boyer_moore): User REGEXP_FASTMAP_MASK for the
bit-and instead of the REGEXP_FASTMAP_SIZE (which is clearly
wrong).

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

build fix from Nelson

* merges:
  Make sure list-load-path-shadows can work after dmp

Coverity and build chain fixes from Nelson

* merges:
  Coverity: Resource leak: CID 400015
  Fix configure.ac
  Fix configure
  xstrncpy saga
  xstrncpy saga
  xstrncpy saga
  Fix tty build broken in last configure.ac changes
  Do not add tty objects when no tty is available
  Coverity: CID 400005: Division or modulo by zero
  Coverity: CID 400006: Division or modulo by zero
  Coverity: Out of bounds write: CID 21140
  Coverity: CID 400012 Overrun dynamic
  Fix build for branches without tags
  Coverity CID 400013 Printf arg type mismatch
  Coverity: CID 400031: Tainted string
  Coverity: CID 400829: Uninit
  Use dup2 if available for -t. Also check for ttyname and isatty in configure.
  Coverity: CID 400828 Resource leak
  Use AS_IF instead of if some places - be more M4sh friendly
  Update copyright, remove unused ent arguments

Make sure list-load-path-shadows can work after dmp

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

Merge branch 'master' of ssh://dio.dreamhost.com/~/repos.nelsonferreira.com/git/sxemacs into for-steve

Coverity: Resource leak: CID 400015

* src/callproc.c (relocate_fd): Make sure that dup'ed file
descriptors < min are closed. In addition, make a better version
when dup2 is available.

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

Merge remote-tracking branch 'njsf/for-steve' into for-steve

Fix configure.ac

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

Fix configure

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

xstrncpy saga

* src/emodules-ng.c (__emodng_open_append_exts): Use xstrncpy instead of strncpy
* src/emodules-ng.c (__emodng_open_prepend_paths): Ditto.
* src/emodules-ng.c (__emodng_open_prepend_paths_append_exts): Ditto.

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

xstrncpy saga

* src/editfns.c (Ftemp_directory): Use xstrncpy instead of strncpy
and xstrncat instead of strcat

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

xstrncpy saga

* src/database/postgresql.c (Fpq_connectdb): Use xstrncpy instead of strncpy
* src/database/postgresql.c (Fpq_connect_start): Ditto.
* src/database/postgresql.c (Fpq_exec): Ditto.
* src/database/postgresql.c (Fpq_get_result): Ditto.

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

Fix tty build broken in last configure.ac changes

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

Merge branch 'for-steve' into njsf-tty

Do not add tty objects when no tty is available

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

Coverity: CID 400005: Division or modulo by zero

* src/media/sound-oss.c (sound_oss_play): Abort early when channels is 0.

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

Coverity: CID 400006: Division or modulo by zero

* src/ui/window.c (Fwindow_displayed_text_pixel_height): Make sure
no division by 0 is attempted and use "best heuristic" of just
adding 3.

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

Coverity: Out of bounds write: CID 21140

* src/search.c (boyer_moore): Make it explicit that the value will
always be within the range of the index. Previously it was a side
effect of the downcast to unsigned char, now its explicit.

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

Coverity: CID 400012 Overrun dynamic

* lib-src/gnuclient.c (clean_string): Avoid coverity aliasing
shortcomings by reusing the same variable where the length is
calculate for the copy pass.

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

Fix build for branches without tags

* autogen.sh (emacs_is_beta): Correct the text for branches
without the release tags...

* lib-src/make-dump-id.c: Include the config.h from the proper
directory. This was not an issue for a long time, since there was
always a tag when under git control. make-dump-id was only
compiled then. Now in autogen.sh we will always make sure there is
a tree version, so this will no longer be an issue again.

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

Merge remote-tracking branch 'njsf/for-steve' into for-steve

Merge remote-tracking branch 'origin/master' into for-steve

Merge branch 'master' of ssh://dio.dreamhost.com/~/repos.nelsonferreira.com/git/sxemacs into for-steve

Merge branch 'master' into njsf-cov

Coverity CID 400013 Printf arg type mismatch

* lib-src/make-path.c: Include string.h so that strerror is
properly defined.

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

Coverity: CID 400031: Tainted string

* lib-src/yow.c (main): Make sure argv[2] is not NULL before copying

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

Coverity: CID 400829: Uninit

* src/media/media.c (Fmedia_stream_aspect): Initialize the
potential return value to nil

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

Use dup2 if available for -t. Also check for ttyname and isatty in configure.

* configure.ac: Add some additional functions to check and add the
HAVE_ISATTY and HAVE_TTYNAME

* src/emacs.c (main_1): Enhance the logging of -t with ttyname if
it is available.

* src/emacs.c (main_1): Use raw_open instead of open to avoid
using the format conversion of the filename, which at this early
stage is not setup yet.

* src/emacs.c (main_1): use dup2 if available to dup the opened
file (and then close the original descriptor). This is much more
reliable than close then open, since dup2 will make sure the
descriptor number will be the desired.

* src/eval.c (grow_specpdl): If grow_specpdl is called very early
on specpdl_size is 0 and will loop infinitely here because 2 * 0
== 0 ;)

* src/sysdep.c (raw_open): New function that just calls the system
open with no argument conversion at all

* src/sysfile.h: declare the new raw_open function.

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

Coverity: CID 400828 Resource leak

* src/emacs.c (main_1): Make sure second dup is fdesc 1

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

xstrncpy updates from Nelson

* merges:
  xstrncpy saga
  xstrncpy saga
  xstrncpy saga
  xstrncpy saga

xstrncpy saga

* src/effi.c (Fffi_load_library): use xstrncpy instead of strncpy, strncat

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

xstrncpy saga

* src/dumper.c (pdump_load): Use xstrncpy instead of strcpy

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

xstrncpy saga

* src/sysdep.c (init_system_name): use xstrncpy instead of strcpy

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

xstrncpy saga

* src/sysdep.c (init_system_name): Use xstrncpy instead of strncpy
* src/sysdep.c (readdir): Ditto.

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

Use AS_IF instead of if some places - be more M4sh friendly

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

Update copyright, remove unused ent arguments

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

Various build updates from Nelson

* merges:
  Add the new silent-rules options to the list of known configure opts
  Warn about ignored tty 'with' options. Fix if with missing fi
  Add additional tty lib 'with' options
  issue warning when no tty nor window system available
  Enable silent building, ie --enable-silent-rules and make V=0
  Correct the remote configuration command message
  new SXE_CHECK_LINK_LIB macro
  Add summary for additional tty libraries
  Rename git-to-steve -> git-for-steve
  xstrncpy saga
  Better checking of string and memory operations. Also, add define FORBID_STRCPY to make runtime asserts on usage of strcpy, strcat and stpcpy.
  xstrncpy saga
  xstrncpy saga
  xstrncpy saga
  xstrncpy saga

Add the new silent-rules options to the list of known configure opts

* m4/sxe-aux.m4 (_SXE_CHECK_PURE_OPT): Add silent-rules options to
list of known configure opts.

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

FFI update from Aidan

* merges:
  Avoid namespace pollution, runtime consing, #'define-ffi-function

Avoid namespace pollution, runtime consing, #'define-ffi-function

Merge remote-tracking branch 'njsf/for-steve' into for-steve

Merge branch 'master' of ssh://dio.dreamhost.com/~/repos.nelsonferreira.com/git/sxemacs into for-steve

Warn about ignored tty 'with' options. Fix if with missing fi

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

Add additional tty lib 'with' options

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

issue warning when no tty nor window system available

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

Enable silent building, ie --enable-silent-rules and make V=0

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