* regex-posix.c (scm_regexp_exec): Add a cast to remove a

[bpt/guile.git] / libguile / ChangeLog

1998-10-15  Jim Blandy  <jimb@zwingli.cygnus.com>

	Warning fixes from Greg Harvey:
	* unif.c (scm_array_set_x): initializer for pos
	* throw.c (scm_ithrow): added initializer for jmpbuf (SCM_UNDEFINED)
	* struct.c (scm_struct_ref, scm_struct_set_x): Added
	initializers for field_type, since EGCS so desparately wants to
	play dumb
	* debug.h (scm_make_gloc, scm_gloc_p, scm_make_iloc, scm_memcons,
	scm_mem_to_proc, scm_proc_to_mem, scm_debug_hang): Added prototypes
	when GUILE_DEBUG is defined.
	* dynwind.h (scm_wind_chain): Same.
	* ports.h (scm_pt_size, scm_pt_member): Same.
	* print.h (scm_current_pstate): Same.
	* procs.h (scm_make_cclo): Same.

1998-10-14  Jim Blandy  <jimb@zwingli.cygnus.com>

	Handle short and long long uniform arrays properly.  (Thanks to
	Clark McGrew.)
	* ramap.c (scm_ra_matchp, scm_array_fill_int, scm_array_index_map_x,
 	raeql_1): Add cases for scm_tc7_svect (short vectors) and
	scm_tc7_llvect (long long vectors).

	Change the way libguile and boot-9.scm are timestamped, to try to
	get rid of these spurious mismatch warnings.  Now both
	libguile/versiondat.h and ice-9/version.scm are generated directly
	by the configuration process, rather than having version.scm
	generated directly, and libguile/versiondat.h generated by the
	Makefile, which is generated by configure.  It seems that
	sometimes the Makefile would change, but versiondat.h depends on
	Makefile.in, not Makefile, so it wouldn't get rebuilt.
	* Makefile.am (versiondat.h): Target removed; this is generated
	directly by the configure script now.
	(BUILT_SOURCES): Remove versiondat.h.
	* versiondat.h.in: New file, transformed by the configure script
	into versiondat.h.
	* Makefile.in: Regenerated.

	* (__scm.h, alist.c, eval.c, feature.c, gsubr.c, numbers.c,
 	options.c): Rename RECKLESS -> SCM_RECKLESS, CAUTIOUS ->
	SCM_CAUTIOUS; this way, 1) there's only one version of each flag
	to define (we used to have both RECKLESS and SCM_RECKLESS), and 2)
	if we want to use them in a header file some day, we can.  (Thanks
	to Michael Livshin.)

	* stime.c (scm_get_internal_real_time): Do the arithmetic using
 	SCM numbers, so we won't have rollover problems; the range of a
 	signed long in milliseconds is about 25 days.  (Thanks to Karl
 	Hegbloom.)

	Don't redefine sleep and usleep.
	* coop.c (sleep, usleep): Remove declarations; we don't use or
	redefine these any more.
	(scm_thread_usleep, scm_thread_sleep): New functions which do the
	job of usleep and sleep in a thread-friendly way.  We can use
	these in the rest of Guile.  Define versions for systems both with
	and without iselect.
	* threads.h (scm_thread_usleep, scm_thread_sleep): New declarations.
	* scmsigs.c (usleep): Clean up oddities declaring usleep; since
 	we're just using it, not redefining it, we can use a K&R style
 	declaration here.
	(sleep): Declare this, too, if the system hasn't.
	(scm_sleep, scm_usleep): Use scm_thread_sleep and
	scm_uthread_sleep if they're available; otherwise, just call the
	system functions.
	* scmconfig.h.in: Regenerated.

	* coop.c (scm_thread_sleep): Make `slept' an unsigned long.

	* coop.c (coop_sleephelp): Remove ANSI #ifdef hair.

1998-10-12  Jim Blandy  <jimb@zwingli.cygnus.com>

	* threads.c: Doc fix.

	The argument type of usleep varies from system to system,
	as does the return type.  We really shouldn't be redefining usleep
	at all, but I don't have time to clean that up before the 1.3.
	release.  It's on the schedule for afterwards.  (Thanks to Julian
	Satchell.)
	* coop.c (usleep): Use USLEEP_ARG_TYPE in prototype and
	definition.
	* scmsigs.c (usleep): Use USLEEP_ARG_TYPE in prototype.
	* scmconfig.h: Regenerated.

	* simpos.c (scm_software_type): Procedure deleted.  This isn't the
	right way to handle system variation.  Autoconf's approach is the
	way and the light.
	* simpos.h (scm_software_type): Declaration deleted.

	* script.c (scm_find_executable): Don't test if unix is #defined
	here; first, NetBSD doesn't #define it, and second, it's the wrong
	way to go about these things.  (Thanks to Perry Metzger.)
	(dld_find_executable): Delete this MSDOS support code.  This isn't
	the way we want to support this; it needs to be rethunk at a
	larger scale.

	* genio.c (scm_do_read_line): Don't just politely check that the
	line was either non-empty or EOF.  Abort if it's empty and not
	EOF.

1998-10-11  Jim Blandy  <jimb@zwingli.cygnus.com>

	* scmconfig.h.in: Regenerated.

	* libguile.h: Don't omit the dynamic linking functions.  (Thanks
	to Greg Badros.)

	* genio.c (scm_do_read_line): Count lines correctly when the file
	doesn't end in a newline.

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

	* genio.c (scm_do_read_line): Maintain the line count correctly.
	(Thanks to Harvey J. Stein and Greg Harvey.)

	* gc.c (scm_return_first): Remove #ifdef __STDC__ garbage; Guile
	requires ANSI now.

	* numbers.c (big2str): Protect t from garbage collection until
	we're done.  (Thanks to Gary Houston.)

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

	* Makefile.am (libguile_la_LDFLAGS): Increment shared library
	version number.
	* Makefile.in: Regenerated.

	* fports.h (scm_setbuf0, scm_setvbuf, scm_setfileno,
	scm_evict_ports, scm_open_file, scm_stdio_to_port): Get rid of
	SCM_P macro.

	Do magic to mix reads and writes on stdio FILE-based ports.
	(Thanks to Christian Lynbech.)
	* fports.c (FPORT_READ_SAFE, FPORT_WRITE_SAFE, FPORT_ALL_OKAY):
	New macros.
	(pre_read, pre_write): New functions.
	(local_fgetc, local_fgets, local_ffwrite, local_fputc,
	local_fputs): Call them.
	(local_fflush): Mark the port as ready for reading and writing.
	(scm_stdio_to_port): Set the FPORT_READ_SAFE, FPORT_WRITE_SAFE
	flags on new port objects.  This might not be accurate --- who
	knows what state the FILE * is in when we get it --- but it won't
	do extraneous calls to fflush or fseek, so it's no worse than the
	behavior before this change.
	* ports.h: Add comment.

	Centralize the creation of port objects based on stdio FILE * in
	fports.c; don't just throw them together anywhere.
	* fports.c (scm_stdio_to_port): Make NAME a SCM value, which is
	what the rest of Guile wants.  Don't set the revealed count;
	that's only appropriate for stdin, stdout, stderr.
	(scm_standard_stream_to_port): This function does set the revealed
	count.
	* init.c (scm_init_standard_ports): Use	scm_standard_stream_to_port,
	not scm_stdio_to_port.
	* filesys.c (scm_open): Call scm_stdio_to_port; don't write it out.
	* fports.c (scm_open_file): Same.
	* posix.c (scm_pipe): Same.
	* socket.c (scm_sock_fd_to_port): Same.
	* ioext.c (scm_fdopen): Same.
	(scm_freopen): Moved from here to...
	* fports.c (scm_freopen): ... here.  This is really something that
	munges the internals of an fport, so it should go here.
	* fports.h (scm_stdio_to_port): Adjust prototype.
	(scm_standard_stream_to_port, scm_freopen): New protoypes.
	* ioext.h (scm_freopen): Prototype removed.

	* filesys.c (set_element, get_element): This can work on both pipe
	and file ports, so use SCM_FPORTP to typecheck, instead of testing
	for scm_tc16_fport.

	* scmconfig.h.in: Regenerated.

	Change the definition of the functions in scm_ptobfuns so that
	they get passed the port object, not the port's stream.
	* ports.h (scm_ptobfuns): Rename all `stream' arguments to `port'.
	* gc.c (scm_gc_sweep): Pass the port itself to the free function.
	* genio.c (scm_putc, scm_puts, scm_lfwrite, scm_fflush, scm_getc):
	Pass the port itself to the scm_ptobs function.
	* ports.c (scm_close_port, scm_force_output, scm_flush_all_ports,
	scm_generic_fgets): Same.
	(putc_void_port, puts_void_port, write_void_port, flush_void_port,
	getc_void_port, fgets_void_port, close_void_port): Just change the
	argument names; these functions don't really do anything.
	* fports.c (local_fgetc, local_fgets, local_fclose, local_fflush,
	local_fputc, local_fputs, local_ffwrite, local_pclose): Take the
	port as an argument, and use SCM_STREAM to get the stdio FILE *.
	Also, use prototyped definitions, and get rid of the extra
	declarations.
	(scm_fptob, scm_pipob): We don't need casts here any more.
	* strports.c (prinstpt): Use prototype declarations.
	(stputc, stwrite, stputs, stgetc): Take the port as an argument,
	and use SCM_STREAM to get the string info.  Also, use prototyped
	definitions, and get rid of the extra declarations.
	* vports.c (sfputc, sfwrite, sfputs, sfflush, sfgetc, sfclose,
	noop0): Same.

	* ports.h (scm_ptobfuns): Replace uses of SCM_P with a straight
	prototype; it's okay (preferred, even!) to use ANSI C in Guile.
	
	* fports.c (local_fgetc, local_fgets): Renamed from scm_fgetc and
	scm_fgets, for consistency.
	(scm_fptop, scm_pipob): References updated.

1998-10-08  Jim Blandy  <jimb@zwingli.cygnus.com>

	Include the source location in error messages for scripts.
	* init.c (scm_boot_guile_1): Use scm_internal_lazy_catch, so the
	stack is still there when we catch the error.
	* throw.c (handler_message): If we are handling an error with a
	message, then put together the proper arguments and call
	scm_display_error, instead of scm_display_error_message.  That
	displays source location, if it can find it.

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

	* gc.c (scm_unprotect_object): Change this so that calls to
	scm_protect_object and scm_unprotect_object nest properly.
	(scm_protect_object): Doc fixes.

	* strings.c (scm_string_set_x): Require the argument to be a
	writable string, not a substring or a symbol.
	* strings.h (SCM_RWSTRINGP, SCM_NRWSTRINGP): New predicates.
	(Thanks to John Redford and Charbel Jacquin.)
	
	* scmconfig.h.in: Regenerated; ../acconfig.h has changed.

1998-10-07  Jim Blandy  <jimb@totoro.red-bean.com>

	* eval.c (safe_setjmp): Remove this misunderstanding.
	(ENTER_APPLY, SCM_CEVAL, SCM_APPLY): Replace with references to
	ordinary setjmp.

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

	* libguile.h: Mark these as C declarations, for compilation by C++
	compilers.

	* snarf.h (SCM_PROC, SCM_PROC1): Remove very odd code in #ifdef
 	__cplusplus clause.  I seriously doubt this ever worked the way
 	the author seems to have intended.

1998-10-05  Jim Blandy  <jimb@zwingli.cygnus.com>

	Utterly needless cleanups to hopelessly messy code.
	* ports.c: Doc fixes.
	(scm_fflush): Moved to ...
	* genio.c (scm_fflush): ... here, amongst all the other port
	method invocation functions.
	* genio.h, ports.h: The prototype moves too.

1998-10-04  Jim Blandy  <jimb@zwingli.cygnus.com>

	* backtrace.c (display_error_body): The current frame does not
 	always have a parent frame; consider a function called directly
 	from the MAIN_FUNC passed to scm_boot_guile.  (Thanks to Maciej
 	Stachowiak.)

	* alloca.c (alloca): Undo yesterday's changes, and simply call
 	malloc directly for storage, and abort if we don't get what we
 	want.  The situation is much simpler --- just call malloc.  Emacs
 	has bizarre/evil requirements (signal handlers might malloc unless
 	you set this global flag, so you have to set the flag around all
 	calls to malloc) which we are certainly not going to conform to,
 	so we can do the simple and obvious thing.

	* coop.c (coop_condition_variable_wait): Make this function
	static.  It's only useful internally --- you should never just
	wait on a condition variable.
	* coop-defs.h (coop_condition_variable_wait): Delete prototype.

1998-10-03  Jim Blandy  <jimb@zwingli.cygnus.com>

	* unif.c (scm_array_set_x): Accept any kind of number as an
	element for a uniform vector of doubles.  This is more consistent
	with Scheme's view of numbers.  (Thanks to Miroslav Silovic.)

	* alloca.c: Use scm_must_malloc to obtain storage.  Hopefully this
	works; I can't conveniently test it myself.  (Thanks to Dvid
	Tillman for the bug report.)

	* init.c: Doc fixes.

	* init.c (invoke_main_func): Load the startup files (boot-9.scm)
 	from here, not from scm_compile_shell_switches (which is a pretty
 	dumb place to do it).
	(scm_load_startup_files): New function.
	(scm_ice_9_already_loaded): Variable moved to here from script.c.
	* script.c (scm_compile_shell_switches): Don't load the startup
	files here.
	(scm_ice_9_already_loaded): Variable moved.
	* init.c (scm_load_startup_files): Prototype for new function.
	* gh_init.c (gh_enter): Doc fix.

1998-10-03    <jimb@savonarola.red-bean.com>

	Some anti-warning changes from Greg Harvey.
	* gh_data.c (gh_scm2doubles): Initialize m, to avoid compiler
	warnings when it doesn't understand our NORETURN declarations in
	error.h.
	* posix.c (scm_mknod): Similar.

1998-10-03  Jim Blandy  <jimb@zwingli.cygnus.com>

	* posix.c (scm_getpwuid): If we can't find an entry, return our
 	own message, instead of using scm_syserror --- the getpwMUMBLE
 	functions don't set `errno' to anything interesting.

1998-10-03    <jimb@savonarola.red-bean.com>

	Get rid of warnings from the cooperative threading system.
	* threads.h (scm_single_thread_p, scm_yield,
	scm_call_with_new_thread, scm_join_thread, scm_make_mutex,
	scm_lock_mutex, scm_unlock_mutex, scm_make_condition_variable,
	scm_wait_condition_variable, scm_signal_condition_variable): Add
	prototypes for these Scheme-visible functions.
	* coop-defs.h (coop_next_runnable_thread,
	coop_wait_for_runnable_thread_now, coop_wait_for_runnable_thread):
	Prototypes for these here, even though they're from iselect.c.
	(coop_condition_variable_wait, coop_join): Add prototypes.
	* coop-threads.c (scm_threads_free_thread, scm_threads_free_mutex,
	scm_threads_free_condvar): Make these smob functions static.
	* coop-threads.h (coop_init): Give this a real prototype.
	* coop.c: #include <unistd.h>, if we have it, for `usleep' and `sleep'.
	(coop_next_runnable_thread): No need to provide prototype; it's in 
	coop-defs.h.

	* scmconfig.h.in: .detarenegeR

	* iselect.c, threads.c: Doc fixes.

1998-10-03  Jim Blandy  <jimb@zwingli.cygnus.com>

	* Makefile.in: Regenerated with a patched automake, to get
	dependency generation right when using EGCS.

	* inet_aton.c (inet_aton): Add prototype, to remove compiler
 	warning. (Thanks to Robert Pluim.)

	* inet_aton.c (inet_aton): Reassure the compiler that the
 	arguments to the <ctype.h> macros are all unsigned characters, not
 	signed characters.

1998-10-03    <jimb@savonarola.red-bean.com>

	Getting rid of more warnings...
	* iselect.c: Test for MISSING_BZERO_DECL, not DECLARE_BZERO; see
	today's change to ../configure.in.
	* scmsigs.c: Test for MISSING_USLEEP_DECL, not DECLARE_USLEEP.
	* scmconfig.h.in: Regenertaded.de.,.__
	* stime.c (strptime): Declare this, #ifdef MISSING_STRPTIME_DECL.
	(scm_localtime, scm_mktime): Use a const char * to manipulate the
	time zone name.

	* readline.c: Doc fix.
	(rl_cleanup_after_signal, rl_free_line_state): Make these static.
	* readline.h (scm_filename_completion_function): Add prototype.
	(scm_init_readline): Make this into a prototype.

	* readline.c (scm_filename_completion_function): Use SCM_PROC to
	declare this, instead of calling scm_make_subr manually.

1998-10-02    <jimb@savonarola.red-bean.com>

	* readline.h (scm_init_readline): Add prototype for this.
	(scm_init_readline): Make this a real prototype.

1998-09-30  Jim Blandy  <jimb@zwingli.cygnus.com>
	
	Warning fixes from Maciej Stachowiak:
	* backtrace.h (scm_display_application, scm_backtrace): Add
	prototypes.
	* debug.c (scm_m_start_stack): Make this function static.
	* fluids.h (scm_fluid_p): Add prototype.
	* procprop.c (scm_i_procedure_arity): Make this function static.
	* regex-posix.c (scm_regexp_error_msg): Make this function static.
	* regex-posix.h (scm_init_regex_posix): Use prototype, not K&R decl.
	* root.h (scm_dynamic_root): Add external prototype.
	* scmsigs.h (scm_usleep): Add external prototype.
	* script.h (scm_init_script): Use prototype, not K&R decl.
	* stacks.h (scm_stack_id): Add external prototype.
	* symbols.h (scm_sysintern0_no_module_lookup): Add external prototype.

1998-09-30  Mark Galassi  <rosalia@cygnus.com>

	* gh.h: took out the definitions of vset and vref, since they are
	replaced by the proper vector routines that correspond to the R4RS
	procedures.

1998-09-29  Jim Blandy  <jimb@totoro.red-bean.com>

	* snarf.h (SCM_VCELL_INIT, SCM_GLOBAL_VCELL_INIT): New macros;
 	these are analogous to SCM_VCELL and SCM_GLOBAL_VCELL but take a
 	third argument, a C expression that should result in a SCM value,
 	which is used to initialize the variable. Reimplemented
 	SCM_CONST_LONG in terms of SCM_VCELL_INIT.  (Thanks to Maciej
	Stachowiak.)

	* version.h (scm_libguile_config_stamp): Add prototype.
	(From Maciej Stachowiak.)

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

	* eval.c (scm_force): Assert that x is SCM_NIMP to fix segv when
	(force 9) is tried.  (Thanks to Karl M. Hegbloom.) 

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

	* print.c (scm_iprin1): Rather than having one i, and using it in
	several places, declare a fresh i local to each block where it is
	used, and give it a signedness appropriate to its use in each case.
	(scm_iprlist): Same.

	* print.c (scm_iprin1): Add cast to avoid unsigned/signed
	comparison warnings.

	* print.c (ENTER_NESTED_DATA): Make i an unsigned long, to avoid
	signed/unsigned clashes.

	* posix.h (scm_tmpnam): Added prototype.

	* 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_readline): Defer interrupts while we're calling
	readline.

	* 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