* print.c (scm_iprin1): Rather than having one i, and using it in

[bpt/guile.git] / libguile / ChangeLog

1998-09-06  Jim Blandy  <jimb@zwingli.cygnus.com>

	* objects.h (scm_set_object_procedure_x, scm_make_class_object,
 	scm_make_subclass_object): Add external prototypes.

	* numbers.c (scm_mkbig): Add cast, and note that signed/unsigned
 	comparison is okay here.

	* numbers.c (scm_istr2int): Add cast; len is known to be positive.

	* numbers.c (scm_bigcomp): Clarify logic, and avoid relying on the
	(true, but confusing) fact that -1 == ((unsigned) 0 - 1).

	* numbers.c (scm_adjbig): Make nsiz an scm_sizet, to avoid mixing
 	signed/unsigned.

	* load.c (swap_port): Make this function static.

	* load.c (scm_search_path): Make max_path_len and max_ext_len
 	unsigned, since they're compared against string sizes.

	* load.c (init_build_info): Make i unsigned.

	* ioext.h (scm_read_line): Add prototype.
	
	* hashtab.c (scm_hash_fn_get_handle, scm_hash_fn_create_handle_x,
 	scm_hash_fn_remove_x): Make hash bucket index local variable k
 	unsigned.  Use scm_ulong2num to pass it to SCM_ASSERT as
 	accurately as possible.

	* gh_data.c (gh_set_substr): Add casts to avoid signed/unsigned
	comparisons, and range checking to make sure those casts are
	harmless.

	* stackchk.h (SCM_STACK_OVERFLOW_P): Change definition to avoid
	signed/unsigned comparisons.

	* smob.c (scm_numsmob): Make this an int, not an scm_sizet, to
	avoid signed/unsigned comparisons.
	* smob.h (scm_numsmob): Change extern declaration to match.

	* ports.c (scm_numptob): Make this an int, not an scm_sizet, to
	avoid signed/unsigned comparisons.
	* ports.h (scm_numptob): Change extern declaration to match.
	(scm_current_load_port, scm_set_port_line_x,
	scm_set_port_column_x): New prototypes.

	* gsubr.c (GSUBR_TEST): Don't #define this.  Nobody's using the
 	test code, and it causes warnings.

	* gh.h (gh_int2scmb, gh_uniform_vector_length,
 	gh_uniform_vector_ref): Added prototypes.

	* Makefile.am (libguile_la_SOURCES): Include Scheme-level
	debugging support unconditionally.  That's backtrace.c, stack.c,
	debug.c, and srcprop.c.
	(EXTRA_libguile_la_SOURCES): Omit those from here.
	* Makefile.in: Regenerated.

1998-08-21  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* options.c (scm_options): Bugfix: Allow empty list of options!

	* debug.c, debug.h (scm_single_step): Removed.
	(scm_with_traps): New procedure.  This procedure could easily be
	written in Scheme but needs to be highly optimized.

	* eval.h, eval.c: New evaluator trap flag: SCM_TRAPS_P.

	* eval.c (SCM_CEVAL, SCM_APPLY): Removed resetting of trap flags.
	Check SCM_TRAPS_P before trapping.

1998-07-30  Jim Blandy  <jimb@zwingli.cygnus.com>

	Changes to avoid signed/unsigned comparison warnings.
	* gc.c (scm_mtrigger, scm_heap_size): Make these unsigned longs.
	(scm_gc_sweep): Make n and j local to the blocks they're used in,
	so they can have appropriate types for each application.  Make i
	signed.  Use initializers in some spots.  I'll probably pay for
	all this tweaking.
	(scm_must_malloc, scm_must_realloc): Use scm_sizet for size args.
	(scm_must_realloc): Make nm unsigned.
	(init_heap_seg): Make new_seg_index and n_new_objects signed.
	(scm_init_storage): Use prototype-style definition, and make the
	argument unsigned.
	* gc.h (scm_heap_size, scm_mtrigger, scm_must_malloc,
	scm_must_realloc, scm_init_storage): Adjust prototype accordingly.
	
	* filesys.c (scm_readlink): Make local vars rv and size ints, to
	avoid signed/unsigned comparison warnings, and because the return
	value of readlink may be -1.  Don't bother casting the third
	argument to readlink.

	* filesys.c (scm_dirname, scm_basename): Move these to their own
	page, at the end of the file.
	* filesys.h (scm_dirname, scm_basename): Add prototypes for these.

	* eval.h (scm_eval_options_interface): Add external prototype for this.
	* eval.c (scm_eval_options_interface): Use prototype-style def'n.
	
	* eval.c (scm_lookupcar1): Make this static.

	* dynl.h (scm_registered_modules, scm_clear_registered_modules):
	Make these prototype declarations, not K&R-style.

	* chars.c (scm_tables_prehistory): Add cast, to remove signed/
 	unsigned comparison warning.

	* appinit.c: File removed.  It had a single function in it, empty,
 	whose reason for existence is explained in no documentation or
 	comment.  I think it's there as a default for some Tcl-style
	initialization, but Tcl abandoned that approach a while ago.
	* Makefile.am (libguile_la_SOURCES): Remove appinit.c.
	(BUILT_SOURCES): Remove appinit.x.
	* Makefile.in: Regenerated.

1998-07-29  Jim Blandy  <jimb@zwingli.cygnus.com>

	* Makefile.in: Regenerated using the last public version of
	automake, not the hacked Cygnus version.

1998-07-28  Jim Blandy  <jimb@zwingli.cygnus.com>

	Remove Totoro kludge.
	* Makefile.in, scmconfig.h.in: Regenerated.
	* init.c, readline.c: Don't check if TOTORO is #defined.

1998-07-26  Jim Blandy  <jimb@zwingli.cygnus.com>

	* Makefile.am: Adjust for new thread configuration system.
	(INCLUDES): Include the value of THREAD_CPPFLAGS.
	(guile_LDADD, check_ldadd): THREAD_LIBS_LOCAL has been renamed from
	THREAD_LIBS.
	(THREAD_LIBS): Definition deleted; automake will generate such
	things automatically.
	* Makefile.in: Regenerated.

1998-07-23  Jim Blandy  <jimb@zwingli.cygnus.com>

	Simplify smob and port marking; set the mark bit in the generic
	marking code, and make marker routines only responsible for
	turning up outgoing pointers.
	* gc.c (scm_gc_mark): Set the mark bit on ports and smobs here,
	before calling the marking function.  Don't call the marking
	function if it's zero.
	* markers.c (scm_mark0): Just return #f.  This function isn't
	necessary at all now, but it's harmless to call it.  We'll leave
	it in so other folks' code doesn't croak at link time.
	(scm_markcdr): Don't call SCM_SETGC8MARK.
	* async.c (mark_async): Don't call SCM_SETGC8MARK.
	* dynl.c (mark_dynl_obj): Same.
	* root.c (mark_root): Same.
	* srcprop.c (marksrcprops): Same.
	* unif.c (markra): Same.
	* variable.c (scm_markvar): Same.
	* ports.c (scm_markstream): Same.
	(void_port_ptob): Specify zero for our marking function.
	* debug.c (debugobjsmob): Same.
	* dynwind.c (guardsmob): Same.
	* filesys.c (dir_smob): Same.
	* fluids.c (fluid_smob): Same.
	* fports.c (scm_fptob, scm_pipob): Same.
	* mallocs.c (mallocsmob): Same.
	* regex-posix.c (regex_t_smob): Same.
	* smob.c (freecell, flob, bigob): Same.
	* threads.c (thread_smob, mutex_smob, condvar_smob): Same.
	* throw.c (jbsmob, lazy_catch_funs): Same.

1998-07-17  Mikael Djurfeldt  <mdj@barbara.nada.kth.se>

	* eval.c (scm_copy_tree): Reverted last change: `eval' uses
	scm_copy_tree on code in order not to let memoized code to leak
	out.  Thus, scm_copy_tree needs to copy vectors as well since
	quasiquote can introduce evaluated code also inside vector
	constants.

1998-07-17  Mikael Djurfeldt  <mdj@barbara.nada.kth.se>

	* eval.c (scm_copy_tree): Removed ability to copy vectors.

	* gh_data.c, gh.h (gh_ints2scm, gh_longs2ivect,
	gh_ulongs2uvect): New procedures.  (Complements gh_doubles2scm and
	gh_doubles2dvect.)

	* unif.c: Say that ivect and uvect are of type signed and unsigned
	long instead of int in commentary so that it correctly describes
	the implementation.

1998-07-12  Mikael Djurfeldt  <mdj@barbara.nada.kth.se>

	* stime.c: Removed declaration of strptime.  (It should be
	declared by the system headers.  If it turns out that some systems
	don't, we'll handle that then.)  (Thanks to Greg Troxel.)

	* stime.h: Renamed TIMEH --> STIMEH

	* backtrace.c (scm_display_error, scm_display_backtrace): In order
	to increase portability, don't use structure assignment.
	(Thanks to Nicolas Neuss.)

	* iselect.c: Use LONG_MAX instead of ULONG_MAX for increased
	portability.
	(finalize_fd_sets): Added empty statement after last case label.
	(Thanks to Nicolas Neuss.)

	* gc.c (scm_igc): Changed //-comment into /*-comment.  (Thanks to
	Nicolas Neuss.)

Sat Jul 11 22:08:21 1998  Mikael Djurfeldt  <mdj@totoro.red-bean.com>

	* init.c, readline.c: OK, I won't have these readline.x bug
 	reports anymore.  We've had them since April.  The current reason
	is a completely unintelligible failure of totoro.red-bean.com to
	do the test for rl_getc_function in libreadline correctly.  This
	kludge overrides the test if we're on totoro so that the snapshot
	generation process can work.

	* readline.c: Define a strdup replacement if not existent on system.

1998-07-12  Mikael Djurfeldt  <mdj@barbara.nada.kth.se>

	* vectors.c, vectors.h (scm_make_vector): Removed third argument.
	This change makes scm_make_vector R5RS compatible.  We cannot keep
	the third argument since people want to be able to deduce the form
	of the C function call only by looking at R5RS.  (At the same time
	we have removed some unnecessary complexity!)

	* eval.c, filesys.c, fluids.c, gc.c, gh_data.c, init.c, kw.c,
	net_db.c, posix.c, print.c, regex-posix.c, scmsigs.c, socket.c,
	stime.c, symbols.c, unif.c, vectors.c, weaks.c: Removed third
	argument in call to scm_make_vector.
	
1998-07-02  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* numbers.h (SCM_NUM2DBL): New macro.  Complements SCM_NUMBERP.
	This macro is useful in applications.

1998-06-21  Mikael Djurfeldt  <mdj@barbara.nada.kth.se>

	* load.c (scm_internal_parse_path): Renamed from scm_parse_path.
	(scm_parse_path, scm_search_path): New Scheme level procedures.

	* load.h (scm_internal_parse_path, scm_parse_path,
	scm_search_path): Declared.

1998-06-18  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* filesys.c (dirname, basename): New procedures.

	* init.c (scm_boot_guile_1): Removed condition around
	scm_init_options.

	* dynwind.c: #include "genio.h"; #include "smob.h"; Implemented a
	new data type (guards) for representation of C level guards and
	data on the wind chain.
	(scm_internal_dynamic_wind): New function.

	* dynwind.h: Declare scm_internal_dynamic_wind.
	
	* root.h (scm_root_state): Added scm_cur_loadp.

	* root.c (mark_root): Added comment about cur_loadp.

	* load.c: #include "dynwind.h";
	(scm_primitive_load): Use scm_inner_dynamic_wind to update
	scm_cur_loadp.

	* init.c (scm_init_standard_ports): Initialize scm_def_loadp.

	* ports.c (current-load-port): New procedure.

1998-06-09  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* ioext.c (scm_isatty_p): Accept any kind of data as argument.  If
	not a tty, return #f.

	* regex-posix.c (scm_regexp_exec): Free malloced memory.  (Thanks
	to Julian Satchell and Roland Kaufmann.)

	* gh.h (gh_memv, gh_member): Fixed typos in macro definitions.
	(gh_write):  Added declaration.  (Thanks to Eiichi Takamori.)

1998-06-07  Mikael Djurfeldt  <mdj@barbara.nada.kth.se>

	* debug.h, debug.c (scm_start_stack): New function.  Implements
	the guts of old scm_m_start_stack.

	* debug.c (scm_m_start_stack): Use scm_start_stack.

	* init.c (scm_start_stack, scm_restart_stack): Renamed to
	start_stack and restart_stack.  (These have static scope.)

1998-05-19  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* readline.c (rl_cleanup_after_signals, rl_free_line_state): New
	readline functions to come in release 2.3.  (Thanks to Chet
	Ramey.)
	(handle_errors): Use the above functions.

1998-05-12  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* readline.c: Improvements for readline support: Handle errors
	better; Implement before-read-hook.

1998-05-11  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* init.c (scm_boot_guile_1), readline.c: Test for
	HAVE_RL_GETC_FUNCTION instead of HAVE_LIBREADLINE.  (Need to
	assure that we have version >= 2.1.)

1998-05-11  Mikael Djurfeldt  <mdj@kenneth>

	* readline.c (scm_add_history): Bugfix: Do strdup before giving
	away the string to add_history.
	(completion_function): Do completion for readline. (Thanks to
	Andrew Archibald.)
	(scm_filename_completion_function): New procedure: Filename
	completer.
	(current_input_getc): New function.  Use this one instead of
	standard getc from readline.

	* throw.c, throw.h (scm_handle_by_throw): New function: This
	handler throws errors to next handler on the dynwind chain.

1998-05-09  Mikael Djurfeldt  <mdj@kenneth>

	* scmsigs.c (scm_usleep): Bugfix: Don't define j if it isn't
	used.

1998-05-03  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* procprop.c (scm_i_procedure_arity): New function.  Returns arity
	of procedure.
	(scm_procedure_properties): Modified to return arity together with
	other properties.
	(scm_procedure_property): Added the read-only property `arity'.
	(scm_set_procedure_property_x): It is an error to set the `arity'
	property.

	* gsubr.h, gsubr.c: Moved macros from gsubr.c to gsubr.h and added
	prefix SCM_; Made f_gsubr_apply global and added prefix scm_.

	* procprop.h (scm_sym_arity): New symbol.

	* objects.c (scm_set_object_procedure_x): New procedure: Use this
	to set the dispatch procedure of an operator or entity object.

	* objects.h (SCM_METACLASS_OPERATOR_LAYOUT, SCM_ENTITY_LAYOUT):
	Made procedure slots read-only.

	* eval.c (SCM_CEVAL): Moved scm_tc7_contin case above
	scm_tcs_cons_gloc case in zero args switch; Fixed args
	construction for operators in scm_tcs_cons_gloc case in two args
	switch.

1998-05-02  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* fluids.c: Removed use of assert.h (in order to avoid
	__eprintf).

	* Makefile.am (libguile_la_LDFLAGS): Added -export-dynamic.

	* dynl.c (maybe_drag_in_eprintf): Disabled through #ifdef 0.

	* eval.c (SCM_CEVAL): Do more thorough argument checking.  This
	change makes the evaluator safer at the cost of evaluation speed.
	It handles the case when the user has added a non-immediate
	improper end of the application form, e.g., `(+ 0 . x)'.
	(Earlier only cases like `(+ 0 . 0)' were handled.)  I've tried to
	minimize the extra cost as much as possible.  The new code is
	enclosed in #ifdef CAUTIOUS regions.  NOTE: This also fixes the
	problem with structs planted directly in the code (e.g. by a
	macro).  This no longer causes segmentation fault.  (Thanks to
	Eric Hanchrow.)
	
	* eval.c, eval.h (scm_eval_args, scm_deval_args): Take one extra
	arg `proc' in order to be able to throw errors; New argument
	checking code.

	* Removed extra #include "debug.h"

1998-04-25  Mikael Djurfeldt  <mdj@kenneth>

	* scmsigs.c: Declare usleep as returning void on some systems.
	(scm_usleep): Return SCM_INUM0 on those systems. (Thanks to Julian
	Satchell.)

	* coop.c (usleep): Return void on some systems.

1998-04-20  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* Makefile.am (libguile_la_LDFLAGS): Removed redundant -rpath.

	* coop.c: Changed return type of usleep to int.

	* scmsigs.c (scm_usleep): New procedure; Declare usleep if it
	isn't found in the OS.

	* iselect.h: #define scm_internal_select select if GUILE_ISELECT
	isn't enabled.  (Thought that I had made this change ages ago...)

	* iselect.c: Declare bzero if not defined by OS.

1998-04-19  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* dynl.c (scm_must_free_argv): Fixed memory leak due to negated
	condition.  (Thanks to John Tobey.)

	* continuations.c (scm_make_cont), debug.c (scm_make_memoized,
	scm_make_debugobj), eval.c (scm_makprom): SCM_DEFER/ALLOW_INTS -->
	A section.

	* __scm.h: Start the long-term project of moving to POSIX threads.
	Phase 1: Classification of all critical sections.
	(SCM_ENTER_A_SECTION, SCM_EXIT_A_SECTION): New macros: Delimiters
	for A sections.  (See comments in __scm.h for details.)

	* dynl.c: Only check that HAVE_DLOPEN is defined before loading
	dynl-dl.c; Test on HAVE_LIBDLD instead of HAVE_DLD.

1998-04-18  Mikael Djurfeldt  <mdj@mdj.nada.kth.se>

	* Makefile.am (BUILT_SOURCES): Added cpp_err_symbols.c,
	cpp_sig_symbols.c, libpath.h and versiondat.h to BUILT_SOURCES
	(libpath.h, versiondat.h): Replaced dependency on Makefile with
	dependencies on $(srcdir)/Makefile.in
	$(top_builddir)/config.status in order to avoid circularity.

	* script.c (scm_compile_shell_switches): Bugfix: Don't discount i
	from argc if argc was 0 initially.

	* Makefile.am (Makefile.am): Replaced THREAD_LIBS --> GUILE_LIBS
	in generation of libpath.h.

1998-04-15  Mikael Djurfeldt  <mdj@nada.kth.se>

	* Makefile.am (libguile_la_LDFLAGS): Bumped version number of
	libguile from 2 to 3.

1998-04-14  Mikael Djurfeldt  <mdj@nada.kth.se>

	* Makefile.am: Added .x-dependencies to variable BUILT_SOURCES.

1998-04-13  Mikael Djurfeldt  <mdj@kenneth>

	* ports.c (scm_port_line, scm_set_port_line_x, scm_port_column,
	scm_set_port_column_x, scm_port_filename,
	scm_set_port_filename_x): Removed optional arguments.  Added
	proper argument checking.

	* eval.c, eval.h, coop.c (scm_eval_stack, SCM_EVAL_STACK): Measure
	stack size in machine words.

	* unif.c (scm_uniform_vector_ref, scm_cvref, scm_array_set_x,
	rapr1): Use SCM_UCHARS instead of SCM_CHARS for strings.  (Thanks