comparison src/font-mgr.c @ 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 a9c41067dd88
children 97eb4942aec8
comparison
equal deleted inserted replaced
5141:0dcd22290039 5142:f965e31a35f0
102 FcPatternDestroy (p->fcpatPtr); 102 FcPatternDestroy (p->fcpatPtr);
103 p->fcpatPtr = 0; 103 p->fcpatPtr = 0;
104 } 104 }
105 } 105 }
106 106
107 static void
108 print_fc_pattern (Lisp_Object obj, Lisp_Object printcharfun,
109 int UNUSED(escapeflag))
110 {
111 struct fc_pattern *c = XFC_PATTERN (obj);
112 if (print_readably)
113 printing_unreadable_object ("#<fc-pattern 0x%x>", c->header.uid);
114 write_fmt_string (printcharfun, "#<fc-pattern 0x%x>", c->header.uid);
115 }
116
117 /* #### We really need an equal method and a hash method (required if you 107 /* #### We really need an equal method and a hash method (required if you
118 have an equal method). For the equal method, we can probably use one 108 have an equal method). For the equal method, we can probably use one
119 or both of 109 or both of
120 110
121 -- Function: FcBool FcPatternEqual (const FcPattern *pa, const 111 -- Function: FcBool FcPatternEqual (const FcPattern *pa, const
142 /* #### nothing here, is this right?? */ 132 /* #### nothing here, is this right?? */
143 { XD_END } 133 { XD_END }
144 }; 134 };
145 135
146 DEFINE_NODUMP_LISP_OBJECT ("fc-pattern", fc_pattern, 136 DEFINE_NODUMP_LISP_OBJECT ("fc-pattern", fc_pattern,
147 0, print_fc_pattern, finalize_fc_pattern, 137 0, external_object_printer, finalize_fc_pattern,
148 0, 0, fcpattern_description, 138 0, 0, fcpattern_description,
149 struct fc_pattern); 139 struct fc_pattern);
150 140
151 /* 141 /*
152 * Helper Functions 142 * Helper Functions
1104 FcConfigDestroy (p->fccfgPtr); 1094 FcConfigDestroy (p->fccfgPtr);
1105 } 1095 }
1106 p->fccfgPtr = 0; 1096 p->fccfgPtr = 0;
1107 } 1097 }
1108 1098
1109 static void
1110 print_fc_config (Lisp_Object obj, Lisp_Object printcharfun,
1111 int UNUSED(escapeflag))
1112 {
1113 struct fc_config *c = XFC_CONFIG (obj);
1114 if (print_readably)
1115 printing_unreadable_object ("#<fc-config 0x%x>", c->header.uid);
1116 write_fmt_string (printcharfun, "#<fc-config 0x%x>", c->header.uid);
1117 }
1118
1119 static const struct memory_description fcconfig_description [] = { 1099 static const struct memory_description fcconfig_description [] = {
1120 /* #### nothing here, is this right?? */ 1100 /* #### nothing here, is this right?? */
1121 { XD_END } 1101 { XD_END }
1122 }; 1102 };
1123 1103
1124 DEFINE_NODUMP_LISP_OBJECT ("fc-config", fc_config, 1104 DEFINE_NODUMP_LISP_OBJECT ("fc-config", fc_config,
1125 0, print_fc_config, finalize_fc_config, 0, 0, 1105 0, external_object_printer, finalize_fc_config,
1126 fcconfig_description, 1106 0, 0, fcconfig_description,
1127 struct fc_config); 1107 struct fc_config);
1128 1108
1129 DEFUN("fc-init", Ffc_init, 0, 0, 0, /* 1109 DEFUN("fc-init", Ffc_init, 0, 0, 0, /*
1130 -- Function: FcBool FcInit (void) 1110 -- Function: FcBool FcInit (void)
1131 Loads the default configuration file and the fonts referenced 1111 Loads the default configuration file and the fonts referenced