Fix minor problems found by static checking.
commit3d5ee10aa258a500e0b70b5eabe9d58cb3ab051e
authorPaul Eggert <eggert@cs.ucla.edu>
Mon, 3 Jun 2013 18:29:30 +0000 (3 11:29 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Mon, 3 Jun 2013 18:29:30 +0000 (3 11:29 -0700)
tree37d030ec632685a294eb351851fdad8ce41800b0
parent068922a2973033ea826b458a17f3e06cf6b44299
Fix minor problems found by static checking.

* data.c (pure_write_error):
Use xsignal2, not Fsignal, as Fsignal might return.
* eval.c (set_backtrace_debug_on_exit): Now static.
(backtrace_p, backtrace_top, backtrace_next, record_in_backtrace):
No longer inline.  EXTERN_INLINE is needed only for functions
defined in .h files.  Reindent function header as per GNU style.
(backtrace_p, backtrace_top, backtrace_next):
Mark EXTERNALLY_VISIBLE so they don't get optimized away by the
compiler or linker.  Add extern decls to pacify gcc -Wall.
* frame.c, frame.h (Qgeometry, Qworkarea, Qmm_size, Qframes, Qsource):
Now static.
* frame.c (free_monitors): Define only on platforms that need it.
* nsterm.m (ns_term_init):
* process.c (catch_child_signal):
Don't worry about whether SIGCHLD is defined, as SIGCHLD is
defined on all porting targets these days.
* process.c, process.h (catch_child_signal):
Make it extern only if NS_IMPL_GNUSTEP is defined.
src/ChangeLog
src/data.c
src/eval.c
src/frame.c
src/frame.h
src/nsterm.m
src/process.c
src/process.h