view lisp/ChangeLog.GTK @ 5169:6c6d78781d59

cleanup of code related to xfree(), better KKCC backtrace capabilities, document XD_INLINE_LISP_OBJECT_BLOCK_PTR, fix some memory leaks, other code cleanup -------------------- ChangeLog entries follow: -------------------- src/ChangeLog addition: 2010-03-24 Ben Wing <ben@xemacs.org> * array.h: * array.h (XD_LISP_DYNARR_DESC): * dumper.c (pdump_register_sub): * dumper.c (pdump_store_new_pointer_offsets): * dumper.c (pdump_reloc_one_mc): * elhash.c: * gc.c (lispdesc_one_description_line_size): * gc.c (kkcc_marking): * lrecord.h: * lrecord.h (IF_NEW_GC): * lrecord.h (enum memory_description_type): * lrecord.h (enum data_description_entry_flags): * lrecord.h (struct opaque_convert_functions): Rename XD_LISP_OBJECT_BLOCK_PTR to XD_INLINE_LISP_OBJECT_BLOCK_PTR and document it in lrecord.h. * data.c: * data.c (finish_marking_weak_lists): * data.c (continue_marking_ephemerons): * data.c (finish_marking_ephemerons): * elhash.c (MARK_OBJ): * gc.c: * gc.c (lispdesc_indirect_count_1): * gc.c (struct): * gc.c (kkcc_bt_push): * gc.c (kkcc_gc_stack_push): * gc.c (kkcc_gc_stack_push_lisp_object): * gc.c (kkcc_gc_stack_repush_dirty_object): * gc.c (KKCC_DO_CHECK_FREE): * gc.c (mark_object_maybe_checking_free): * gc.c (mark_struct_contents): * gc.c (mark_lisp_object_block_contents): * gc.c (register_for_finalization): * gc.c (mark_object): * gc.h: * lisp.h: * profile.c: * profile.c (mark_profiling_info_maphash): Clean up KKCC code related to DEBUG_XEMACS. Rename kkcc_backtrace() to kkcc_backtrace_1() and add two params: a `size' arg to control how many stack elements to print and a `detailed' arg to control whether Lisp objects are printed using `debug_print()'. Create front-ends to kkcc_backtrace_1() -- kkcc_detailed_backtrace(), kkcc_short_backtrace(), kkcc_detailed_backtrace_full(), kkcc_short_backtrace_full(), as well as shortened versions kbt(), kbts(), kbtf(), kbtsf() -- to call it with various parameter values. Add an `is_lisp' field to the stack and backtrace structures and use it to keep track of whether an object pushed onto the stack is a Lisp object or a non-Lisp structure; in kkcc_backtrace_1(), don't try to print a non-Lisp structure as a Lisp object. * elhash.c: * extents.c: * file-coding.c: * lrecord.h: * lrecord.h (IF_NEW_GC): * marker.c: * marker.c (Fmarker_buffer): * mule-coding.c: * number.c: * rangetab.c: * specifier.c: New macros IF_OLD_GC(), IF_NEW_GC() to simplify declaration of Lisp objects when a finalizer may exist in one but not the other. Use them appropriately. * extents.c (finalize_extent_info): Don't zero out data->soe and data->extents before trying to free, else we get memory leaks. * lrecord.h (enum lrecord_type): Make the first lrecord type have value 1 not 0 so that 0 remains without implementation and attempts to interpret zeroed memory as a Lisp object will be more obvious. * array.c (Dynarr_free): * device-msw.c (msprinter_delete_device): * device-tty.c (free_tty_device_struct): * device-tty.c (tty_delete_device): * dialog-msw.c (handle_directory_dialog_box): * dialog-x.c: * emacs.c (free_argc_argv): * emodules.c (attempt_module_delete): * file-coding.c (chain_finalize_coding_stream_1): * file-coding.c (chain_finalize_coding_stream): * glyphs-eimage.c: * glyphs-eimage.c (jpeg_instantiate_unwind): * glyphs-eimage.c (gif_instantiate_unwind): * glyphs-eimage.c (png_instantiate_unwind): * glyphs-eimage.c (tiff_instantiate_unwind): * imgproc.c: * imgproc.c (build_EImage_quantable): * insdel.c (uninit_buffer_text): * mule-coding.c (iso2022_finalize_detection_state): * objects-tty.c (tty_finalize_color_instance): * objects-tty.c (tty_finalize_font_instance): * objects-tty.c (tty_font_list): * process.c: * process.c (finalize_process): * redisplay.c (add_propagation_runes): * scrollbar-gtk.c: * scrollbar-gtk.c (gtk_free_scrollbar_instance): * scrollbar-gtk.c (gtk_release_scrollbar_instance): * scrollbar-msw.c: * scrollbar-msw.c (mswindows_free_scrollbar_instance): * scrollbar-msw.c (unshow_that_mofo): * scrollbar-x.c (x_free_scrollbar_instance): * scrollbar-x.c (x_release_scrollbar_instance): * select-x.c: * select-x.c (x_handle_selection_request): * syntax.c: * syntax.c (uninit_buffer_syntax_cache): * text.h (eifree): If possible, whenever we call xfree() on a field in a structure, set the field to 0 afterwards. A lot of code is written so that it checks the value being freed to see if it is non-zero before freeing it -- doing this and setting the value to 0 afterwards ensures (a) we won't try to free twice if the cleanup code is called twice; (b) if the object itself stays around, KKCC won't crash when attempting to mark the freed field. * rangetab.c: Add a finalization method when not NEW_GC to avoid memory leaks. (#### We still get memory leaks when NEW_GC; need to convert gap array to Lisp object).
author Ben Wing <ben@xemacs.org>
date Wed, 24 Mar 2010 01:22:51 -0500
parents 0784d089fdc9
children aa729daae5e2
line wrap: on
line source

2000-09-12  William M. Perry  <wmperry@aventail.com>

	* dialog-gtk.el (popup-builtin-open-dialog): Went back to
	using our lisp implementation of the file dialog.  Much more
	featureful.

	* ui/gtk-file-dialog.el: Reworked to use CList instead of Tree
	elements (more like the `real' GTK file selector.

2000-09-10  William M. Perry  <wmperry@aventail.com>

	* gtk-iso8859-1.el (gtk-iso8859-1): Need to actually provide
	the feature

2000-09-09  William M. Perry  <wmperry@aventail.com>

	* dialog-gtk.el (popup-builtin-open-dialog): Guard against
	calling gtk-main-quit too many times when destroying the
	file-selection dialog.

2000-09-08  William M. Perry  <wmperry@aventail.com>

	* gtk-init.el (gtk-initialize-compose): Initialize the compose
	map like X does.

2000-09-03  William M. Perry  <wmperry@aventail.com>

	* menubar-items.el (default-menubar): Include the font & size
	menus when running under GTK.

	* gtk-faces.el (x-font-regexp-*): Added variable aliases for
	the x-font-regexp-* variables.  x-font-menu works now.

	* x-font-menu.el (font-menu-set-font): When setting the font, make
	sure we don't set the type to 'x' blithely.  This code is shared
	with GTK now.

2000-08-30  William M. Perry  <wmperry@aventail.com>

	* dialog-gtk.el (popup-builtin-open-dialog): Signal 'quit' if the
	user hits the cancel button.  This gets rid of the 'wrong type
	argument: stringp, nil' error.

2000-08-28  William M. Perry  <wmperry@aventail.com>

	* menubar-items.el (default-menubar): Disable the GTK font menu
	item.

	* dialog-gtk.el (popup-builtin-open-dialog): Reimplemented the
	file-open dialog to use the normal GTK selector.

2000-07-26  William M. Perry  <wmperry@aventail.com>

	* gtk-init.el (init-post-gtk-win): Define the mule-fonts specifier
	tag and default fonts for it when mule is provided.  This will
	make x-symbol.el work.

2000-07-24  William M. Perry  <wmperry@aventail.com>

	* ui/gdk.el: Ditto.

	* ui/gnome-widgets.el: Ditto.

	* ui/gtk-widgets.el: Updated all gtk-import-function calls to pass
	a symbol instead of a quoted string.

	* ui/gtk-ffi.el (gtk-import-variable): Make gtk-import-variable
	able to take symbols instead of just strings.  More consistent
	with gtk-import-function this way.

2000-07-22  William M. Perry  <wmperry@aventail.com>

	* gtk-init.el (init-pre-gtk-win): Did not realize I had to do lisp
	hackery to get '-unmapped' to work.

2000-07-12  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-widgets.el (gtk-ctree-post-recursive): 
	(gtk-ctree-post-recursive-to-depth): 
	(gtk-ctree-pre-recursive): 
	(gtk-ctree-pre-recursive-to-depth): Added wrappers around the
	combined gtk-ctree-recurse to make things easier on GTK authors
	porting other code.

2000-07-11  William M. Perry  <wmperry@aventail.com>

	* gtk-init.el (init-gtk-win): Set gtk-initial-geometry out of
	command-line-args-left.
	(gtk-filter-arguments): New function to filter out only GTK/GNOME
	approved command line arguments.
	(init-gtk-win): Set gtk-initial-argv-list by filtering it.  This
	way we get session management/etc from GNOME.

2000-07-07  William M. Perry  <wmperry@aventail.com>

	* gtk-faces.el (gtk-init-global-faces): Make sure to pass in a GTK
	device (any GTK device) try-font-name when initializing the global
	faces or it gets confused and cannot find a font, so the code in
	faces.el ends up setting device-specific faces, which are hard (or
	at least non-obvious) for users to work around.

2000-07-01  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-widgets.el (GtkType): Added GtkCTree finally.

	* ui/gtk-ffi.el (gtk-ffi-check-function): New function that will
	try to call a newly defined FFI function and report on whether we
	need to define any new marshallers for it.
	(gtk-ffi-debug): If non-nil, then we will check ALL functions that
	come through gtk-import-function.  All existing imported functions
	have been checked, and a few missings ones were added.

	* ui/glade.el: New file to import libglade functions.

2000-06-30  William M. Perry  <wmperry@aventail.com>

	* ui/gdk.el: Added most of the GDK drawing primitives.

2000-06-27  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-widgets.el: Import gtk-major-version, gtk-minor-version,
	gtk-micro-version, gtk-interface-age, and gtk-binary-age.

	* ui/gtk-ffi.el (gtk-import-variable): New macro to import a
	variable.  Needed to do it as a function, otherwise you could not
	byte compile / dump the file in a non-windowed XEmacs.

2000-06-23  William M. Perry  <wmperry@aventail.com>

	* menubar-items.el (default-menubar): Disable
	make-frame-on-display if the function is not available.

2000-06-02  William M. Perry  <wmperry@aventail.com>

	* gtk-mouse.el (gtk-init-pointers): Make sure we set a toolbar
	pointer.  Looked kind of silly to have the 'xterm' cursor in
	there by default.

2000-06-01  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-widgets.el: Imported GtkStatusbar

	* ui/gtk-marshal.el (define-marshaller): All marshalling functions
	are now static.
	* ui/gtk-marshal.el: Now outputs a function to populate a
	hashtable with mappings from function name -> function pointer.
	Also emits the find_marshaller () function that looks at this
	hashtable.

2000-05-29  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-ffi.el (gtk-import-function): Allow passing in of a
	symbol for the function name, as well as a string.

	* ui/gtk-widgets.el: Import the GtkSpinButton widget.

2000-05-26  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-ffi.el (gtk-import-function): Rewrote as a macro so that
	teh file can be safely byte-compiled.  Argument values no longer
	need to be quoted, and the function is not actually imported until
	the function is called.  Should save even more on loadup time.
	(gtk-import-function): Do not defvar `lisp-name' - put the FFI
	object on the symbols plist.  Makes the lisp variable namespace
	that much cleaner.

	* ui/gtk-widgets.el: New uber-file containing all the GTK imported
	functions.  Load time is significantly faster than requiring ~90
	different (usually 5 line) .el files, and polluting the 'features'
	variable.

2000-05-23  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-widget-accessors.el (define-widget-accessors): Fixed
	bogus handling of GTkListOfString and GtkListOfObject slots.
	These need to have the FULL type, not the fundamental type.  And
	_POINTER was just plain wrong for them - cause beautiful crashes
	on linux. :)

	* gtk-faces.el (gtk-choose-font): New function to change fonts
	based on a GTK font selection dialog.

	* menubar-items.el (default-menubar): Only show the font/size
	submenus when we are in an X frame.
	(default-menubar): Show a gtk-specific item when on GTK frames.

	* ui/gtk-marshal.el (define-marshaller): We need to special case
	anything with FLOAT in the argument list or the parameters get
	screwed up royally.

2000-05-21  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-file-dialog.el (gtk-file-dialog-fill-file-list): New file
	dialog that is actually useful and much prettier than the default
	GTK one.

2000-05-20  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-ffi.el (gtk-import-function): Auto-run gnome-*-get-type
	routines as well as the gtk-*-get-type ones.

	* minibuf.el (mouse-read-file-name-1): Now tries to use the new
	builtin dialog spec ben wrote about.  Uses the GTK file selection
	dialog.  We should be able to come up with something much sexier
	though - the default dialog box for GTK sucks hard.

2000-05-17  William M. Perry  <wmperry@aventail.com>

	* gtk-faces.el (gtk-init-device-faces): Make use of the extended
	return values of gtk-style-info.  Set the 'highlight' face to look
	like GTK_STATE_PRELIGHT and 'zmacs-region' to be
	GTK_STATE_SELECTED.  Unfortunately these two faces will not
	automatically be updated because they are not exposed to lisp
	like Vdefault_face and friends.

2000-05-16  William M. Perry  <wmperry@aventail.com>

	* gtk-faces.el (gtk-init-device-faces): Removed a whole bunch of
	face munging that is now done down in the guts of the GtkXEmacs
	widget.

	* gnuserv.el (gnuserv-edit-files): Handle GTK devices.

	* ui/gtk-ffi.el (gtk-import-function): Make this a noop if
	noninteractive.  This allows us to compile the files during the
	make process.

2000-05-10  William M. Perry  <wmperry@aventail.com>

	* dialog-gtk.el: New file implementing popup dialogs in Lisp using
	GTK primitives.  Called from dialog-gtk.c

	* dumped-lisp.el (preloaded-file-list): Make sure we load up
	menubar-items under GTK.
	(preloaded-file-list): Load up dialog-gtk when using GTK.

2000-05-08  William M. Perry  <wmperry@aventail.com>

	* ui/gtk-widget-accessors.el (define-widget-accessors): New file
	to define C functions that go into
	../../src/emacs-widget-accessors.c.  This is a hack to get around
	the lack of accessor/settor functions in GTK for a LOT of things
	that are required for full functionality (like dialogs)

2000-05-07  William M. Perry  <wmperry@aventail.com>

	* gtk-faces.el (gtk-init-face-from-resources): Set the highlight
	face as well.