comparison src/ChangeLog @ 5142:f965e31a35f0

reduce lcrecord headers to 2 words, rename printing_unreadable_object -------------------- ChangeLog entries follow: -------------------- man/ChangeLog addition: 2010-03-13 Ben Wing <ben@xemacs.org> * internals/internals.texi (Working with Lisp Objects): * internals/internals.texi (Writing Macros): * internals/internals.texi (lrecords): More rewriting to correspond with changes from *LRECORD* to *LISP_OBJECT*. modules/ChangeLog addition: 2010-03-13 Ben Wing <ben@xemacs.org> * postgresql/postgresql.c (print_pgconn): * postgresql/postgresql.c (print_pgresult): printing_unreadable_object -> printing_unreadable_object_fmt. 2010-03-13 Ben Wing <ben@xemacs.org> * ldap/eldap.c (print_ldap): printing_unreadable_object -> printing_unreadable_object_fmt. src/ChangeLog addition: 2010-03-13 Ben Wing <ben@xemacs.org> * alloc.c (alloc_sized_lrecord_1): * alloc.c (alloc_sized_lrecord_array): * alloc.c (old_alloc_sized_lcrecord): * alloc.c (disksave_object_finalization_1): * alloc.c (mark_lcrecord_list): * alloc.c (alloc_managed_lcrecord): * alloc.c (free_managed_lcrecord): * alloc.c (tick_lcrecord_stats): * alloc.c (sweep_lcrecords_1): * buffer.c (print_buffer): * buffer.c (DEFVAR_BUFFER_LOCAL_1): * casetab.c: * casetab.c (print_case_table): * console.c (print_console): * console.c (DEFVAR_CONSOLE_LOCAL_1): * data.c (print_weak_list): * data.c (print_weak_box): * data.c (print_ephemeron): * data.c (ephemeron_equal): * database.c (print_database): * database.c (finalize_database): * device-msw.c (sync_printer_with_devmode): * device-msw.c (print_devmode): * device-msw.c (finalize_devmode): * device.c: * device.c (print_device): * elhash.c: * elhash.c (print_hash_table): * eval.c (print_subr): * eval.c (print_multiple_value): * event-stream.c (event_stream_resignal_wakeup): * events.c (clear_event_resource): * events.c (zero_event): * events.c (print_event): * extents.c: * extents.c (print_extent): * file-coding.c (print_coding_system): * font-mgr.c: * font-mgr.c (Ffc_init): * frame.c: * frame.c (print_frame): * gc.c: * gc.c (GC_CHECK_NOT_FREE): * glyphs.c: * glyphs.c (print_image_instance): * glyphs.c (print_glyph): * gui.c (print_gui_item): * gui.c (copy_gui_item): * keymap.c (print_keymap): * keymap.c (MARKED_SLOT): * lisp.h: * lisp.h (struct Lisp_String): * lisp.h (DEFUN): * lisp.h (DEFUN_NORETURN): * lrecord.h: * lrecord.h (NORMAL_LISP_OBJECT_UID): * lrecord.h (struct lrecord_header): * lrecord.h (set_lheader_implementation): * lrecord.h (struct old_lcrecord_header): * lrecord.h (struct free_lcrecord_header): * marker.c (print_marker): * mule-charset.c: * mule-charset.c (print_charset): * objects.c (print_color_instance): * objects.c (print_font_instance): * objects.c (finalize_font_instance): * print.c (print_cons): * print.c (printing_unreadable_object_fmt): * print.c (printing_unreadable_lisp_object): * print.c (external_object_printer): * print.c (internal_object_printer): * print.c (debug_p4): * print.c (ext_print_begin): * process.c (print_process): * rangetab.c (print_range_table): * rangetab.c (range_table_equal): * scrollbar.c (free_scrollbar_instance): * specifier.c (print_specifier): * specifier.c (finalize_specifier): * symbols.c (guts_of_unbound_marker): * symeval.h: * symeval.h (DEFVAR_SYMVAL_FWD): * tooltalk.c: * tooltalk.c (print_tooltalk_message): * tooltalk.c (print_tooltalk_pattern): * ui-gtk.c (ffi_object_printer): * ui-gtk.c (emacs_gtk_object_printer): * ui-gtk.c (emacs_gtk_boxed_printer): * window.c (print_window): * window.c (free_window_mirror): * window.c (debug_print_window): * xemacs.def.in.in: (1) printing_unreadable_object -> printing_unreadable_object_fmt. (2) printing_unreadable_lcrecord -> printing_unreadable_lisp_object and fix up so it no longer requires an lcrecord. These previous changes eliminate most of the remaining places where the terms `lcrecord' and `lrecord' occurred outside of specialized code. (3) Fairly major change: Reduce the number of words in an lcrecord from 3 to 2. The third word consisted of a uid that duplicated the lrecord uid, and a single free bit, which was moved into the lrecord structure. This reduces the size of the `uid' slot from 21 bits to 20 bits. Arguably this isn't enough -- we could easily have more than 1,000,000 or so objects created in a session. The answer is (a) It doesn't really matter if we overflow the uid field because it's only used for debugging, to identify an object uniquely (or pretty much so). (b) If we cared about it overflowing and wanted to reduce this, we could make it so that cons, string, float and certain other frob-block types that never print out the uid simply don't store a uid in them and don't increment the lrecord_uid_counter. (4) In conjunction with (3), create new macro NORMAL_LISP_OBJECT_UID() and use it to abstract out the differences between NEWGC and old-GC in accessing the `uid' value from a "normal Lisp Object pointer". (5) In events.c, use zero_nonsized_lisp_object() in place of custom- written equivalent. In font-mgr.c use external_object_printer() in place of custom-written equivalents.
author Ben Wing <ben@xemacs.org>
date Sat, 13 Mar 2010 05:38:08 -0600
parents 0dcd22290039
children 186aebf7f6c6
comparison
equal deleted inserted replaced
5141:0dcd22290039 5142:f965e31a35f0
1 2010-03-13 Ben Wing <ben@xemacs.org>
2
3 * alloc.c (alloc_sized_lrecord_1):
4 * alloc.c (alloc_sized_lrecord_array):
5 * alloc.c (old_alloc_sized_lcrecord):
6 * alloc.c (disksave_object_finalization_1):
7 * alloc.c (mark_lcrecord_list):
8 * alloc.c (alloc_managed_lcrecord):
9 * alloc.c (free_managed_lcrecord):
10 * alloc.c (tick_lcrecord_stats):
11 * alloc.c (sweep_lcrecords_1):
12 * buffer.c (print_buffer):
13 * buffer.c (DEFVAR_BUFFER_LOCAL_1):
14 * casetab.c:
15 * casetab.c (print_case_table):
16 * console.c (print_console):
17 * console.c (DEFVAR_CONSOLE_LOCAL_1):
18 * data.c (print_weak_list):
19 * data.c (print_weak_box):
20 * data.c (print_ephemeron):
21 * data.c (ephemeron_equal):
22 * database.c (print_database):
23 * database.c (finalize_database):
24 * device-msw.c (sync_printer_with_devmode):
25 * device-msw.c (print_devmode):
26 * device-msw.c (finalize_devmode):
27 * device.c:
28 * device.c (print_device):
29 * elhash.c:
30 * elhash.c (print_hash_table):
31 * eval.c (print_subr):
32 * eval.c (print_multiple_value):
33 * event-stream.c (event_stream_resignal_wakeup):
34 * events.c (clear_event_resource):
35 * events.c (zero_event):
36 * events.c (print_event):
37 * extents.c:
38 * extents.c (print_extent):
39 * file-coding.c (print_coding_system):
40 * font-mgr.c:
41 * font-mgr.c (Ffc_init):
42 * frame.c:
43 * frame.c (print_frame):
44 * gc.c:
45 * gc.c (GC_CHECK_NOT_FREE):
46 * glyphs.c:
47 * glyphs.c (print_image_instance):
48 * glyphs.c (print_glyph):
49 * gui.c (print_gui_item):
50 * gui.c (copy_gui_item):
51 * keymap.c (print_keymap):
52 * keymap.c (MARKED_SLOT):
53 * lisp.h:
54 * lisp.h (struct Lisp_String):
55 * lisp.h (DEFUN):
56 * lisp.h (DEFUN_NORETURN):
57 * lrecord.h:
58 * lrecord.h (NORMAL_LISP_OBJECT_UID):
59 * lrecord.h (struct lrecord_header):
60 * lrecord.h (set_lheader_implementation):
61 * lrecord.h (struct old_lcrecord_header):
62 * lrecord.h (struct free_lcrecord_header):
63 * marker.c (print_marker):
64 * mule-charset.c:
65 * mule-charset.c (print_charset):
66 * objects.c (print_color_instance):
67 * objects.c (print_font_instance):
68 * objects.c (finalize_font_instance):
69 * print.c (print_cons):
70 * print.c (printing_unreadable_object_fmt):
71 * print.c (printing_unreadable_lisp_object):
72 * print.c (external_object_printer):
73 * print.c (internal_object_printer):
74 * print.c (debug_p4):
75 * print.c (ext_print_begin):
76 * process.c (print_process):
77 * rangetab.c (print_range_table):
78 * rangetab.c (range_table_equal):
79 * scrollbar.c (free_scrollbar_instance):
80 * specifier.c (print_specifier):
81 * specifier.c (finalize_specifier):
82 * symbols.c (guts_of_unbound_marker):
83 * symeval.h:
84 * symeval.h (DEFVAR_SYMVAL_FWD):
85 * tooltalk.c:
86 * tooltalk.c (print_tooltalk_message):
87 * tooltalk.c (print_tooltalk_pattern):
88 * ui-gtk.c (ffi_object_printer):
89 * ui-gtk.c (emacs_gtk_object_printer):
90 * ui-gtk.c (emacs_gtk_boxed_printer):
91 * window.c (print_window):
92 * window.c (free_window_mirror):
93 * window.c (debug_print_window):
94 * xemacs.def.in.in:
95 (1) printing_unreadable_object -> printing_unreadable_object_fmt.
96 (2) printing_unreadable_lcrecord -> printing_unreadable_lisp_object
97 and fix up so it no longer requires an lcrecord.
98
99 These previous changes eliminate most of the remaining places where
100 the terms `lcrecord' and `lrecord' occurred outside of specialized
101 code.
102
103 (3) Fairly major change: Reduce the number of words in an lcrecord
104 from 3 to 2. The third word consisted of a uid that duplicated the
105 lrecord uid, and a single free bit, which was moved into the lrecord
106 structure. This reduces the size of the `uid' slot from 21 bits to
107 20 bits. Arguably this isn't enough -- we could easily have more than
108 1,000,000 or so objects created in a session. The answer is
109 (a) It doesn't really matter if we overflow the uid field because
110 it's only used for debugging, to identify an object uniquely
111 (or pretty much so).
112 (b) If we cared about it overflowing and wanted to reduce this,
113 we could make it so that cons, string, float and certain other
114 frob-block types that never print out the uid simply don't
115 store a uid in them and don't increment the lrecord_uid_counter.
116
117 (4) In conjunction with (3), create new macro NORMAL_LISP_OBJECT_UID()
118 and use it to abstract out the differences between NEWGC and old-GC
119 in accessing the `uid' value from a "normal Lisp Object pointer".
120
121 (5) In events.c, use zero_nonsized_lisp_object() in place of custom-
122 written equivalent. In font-mgr.c use external_object_printer()
123 in place of custom-written equivalents.
1 2010-03-07 Ben Wing <ben@xemacs.org> 124 2010-03-07 Ben Wing <ben@xemacs.org>
2 125
3 * number.c (bignum_finalize): 126 * number.c (bignum_finalize):
4 * number.c (ratio_finalize): 127 * number.c (ratio_finalize):
5 * number.c (bigfloat_finalize): 128 * number.c (bigfloat_finalize):