annotate tests/tooltalk/load-file.c @ 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 131b0175ea99
children 9fc91aa3a927
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
70
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
1 /* load-file.c - load & execute an Emacs Lisp file in XEmacs using ToolTalk */
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
2
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
3 /* See `tooltalk-load-file-handler' in the file lisp/tooltalk/tooltalk-init.el
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
4 for the receiver side. */
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
5
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
6 #include <desktop/tt_c.h>
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
7 #include <stdio.h>
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
8 #include <stdlib.h>
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
9 #include <sys/wait.h>
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
10
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
11 Tt_status tter;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
12
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
13 #define exit_err_ptr(ptr) \
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
14 if ((tter = tt_ptr_error(ptr)) != TT_OK) \
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
15 { fprintf(stderr, "%d:%s\n", __LINE__, tt_status_message(tter)); exit(1); }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
16
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
17 #define exit_err(stat) \
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
18 if ((tter = stat) != TT_OK) \
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
19 { fprintf(stderr, "%d:%s\n", __LINE__, tt_status_message(tter)); exit(1); }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
20
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
21 Tt_callback_action callback_fn(Tt_message msg, Tt_pattern pat);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
22 static Tt_message create_new_message(char *pathname);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
23 static int initialize_tooltalk(void);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
24 static void usage(void);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
25
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
26 static char* tt_procid;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
27
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
28 Tt_callback_action
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
29 callback_fn(Tt_message msg, Tt_pattern pat)
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
30 {
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
31 tt_message_destroy(msg);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
32 return TT_CALLBACK_PROCESSED;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
33 }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
34
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
35 static Tt_message
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
36 create_new_message(char *pathname)
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
37 {
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
38 Tt_message msg;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
39
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
40 msg = tt_message_create();
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
41 exit_err_ptr(msg);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
42
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
43 exit_err(tt_message_address_set (msg, TT_PROCEDURE));
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
44 exit_err(tt_message_class_set (msg, TT_REQUEST));
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
45 exit_err(tt_message_scope_set (msg, TT_SESSION));
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
46 exit_err(tt_message_op_set (msg, "emacs-load-file"));
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
47 exit_err(tt_message_arg_add (msg, TT_IN, "file", pathname));
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
48 exit_err(tt_message_callback_add(msg, callback_fn));
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
49
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
50 return msg;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
51 }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
52
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
53 static int
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
54 initialize_tooltalk(void) {
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
55 int rcode;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
56
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
57 tt_procid = tt_open();
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
58 if ((rcode = tt_ptr_error(tt_procid)) != TT_OK) {
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
59 return rcode;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
60 }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
61
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
62 if ((rcode = tt_session_join(tt_default_session())) != TT_OK) {
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
63 return (rcode);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
64 }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
65 }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
66
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
67
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
68 static void
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
69 usage(void)
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
70 {
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
71 fprintf(stderr,
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
72 "Usage: load-file pathname\n\n");
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
73 }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
74
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
75 void
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
76 main(argc, argv)
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
77 int argc;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
78 char *argv[];
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
79 {
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
80
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
81 Tt_message msg;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
82
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
83 if (argc != 2) {
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
84 usage();
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
85 exit(0);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
86 }
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
87
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
88 exit_err(initialize_tooltalk());
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
89
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
90 msg = create_new_message(argv[1]);
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
91
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
92 exit_err(tt_message_send(msg));
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
93
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
94 return;
131b0175ea99 Import from CVS: tag r20-0b30
cvs
parents:
diff changeset
95 }