Mercurial > hg > xemacs-beta
diff src/redisplay.c @ 5038:9410323e4b0d
major dynarr fixes
-------------------- ChangeLog entries follow: --------------------
src/ChangeLog addition:
2010-02-20 Ben Wing <ben@xemacs.org>
* device-x.c (Fx_get_resource):
* dynarr.c:
* dynarr.c (Dynarr_realloc):
* dynarr.c (Dynarr_newf):
* dynarr.c (Dynarr_lisp_realloc):
* dynarr.c (Dynarr_lisp_newf):
* dynarr.c (Dynarr_resize):
* dynarr.c (Dynarr_insert_many):
* dynarr.c (Dynarr_delete_many):
* dynarr.c (Dynarr_memory_usage):
* dynarr.c (stack_like_free):
* file-coding.c (coding_reader):
* file-coding.c (gzip_convert):
* gutter.c (output_gutter):
* lisp.h:
* lisp.h (Dynarr_declare):
* lisp.h (DYNARR_SET_LISP_IMP):
* lisp.h (CHECK_NATNUM):
* profile.c (create_timing_profile_table):
* redisplay-output.c (sync_rune_structs):
* redisplay-output.c (sync_display_line_structs):
* redisplay-output.c (redisplay_output_window):
* redisplay.c:
* redisplay.c (get_display_block_from_line):
* redisplay.c (add_ichar_rune_1):
* redisplay.c (ensure_modeline_generated):
* redisplay.c (generate_displayable_area):
* redisplay.c (regenerate_window):
* redisplay.c (update_line_start_cache):
* signal.c:
* signal.c (check_quit):
Lots of rewriting of dynarr code.
(1) Lots of documentation added. Also fix places that
referenced a now-bogus internals node concerning redisplay
critical sections.
(2) Rename:
Dynarr_add_lisp_string -> Dynarr_add_ext_lisp_string
Dynarr_set_length -> Dynarr_set_lengthr ("restricted")
Dynarr_increment -> Dynarr_incrementr
Dynarr_resize_if -> Dynarr_resize_to_add
(3) New functions:
Dynarr_elsize = dy->elsize_
Dynarr_set_length(): Set length, resizing as necessary
Dynarr_set_length_and_zero(): Set length, resizing as necessary,
zeroing out new elements
Dynarr_increase_length(), Dynarr_increase_length_and_zero():
Optimization of Dynarr_set_length(), Dynarr_set_length_and_zero()
when size is known to increase
Dynarr_resize_to_fit(): Resize as necessary to fit a given length.
Dynarr_set(): Set element at a given position, increasing length
as necessary and setting any newly created positions to 0
(4) Use Elemcount, Bytecount.
(5) Rewrite many macros as inline functions.
author | Ben Wing <ben@xemacs.org> |
---|---|
date | Sat, 20 Feb 2010 03:46:22 -0600 |
parents | 16112448d484 |
children | 548f1f47eb82 |
line wrap: on
line diff
--- a/src/redisplay.c Sat Feb 20 03:45:15 2010 -0600 +++ b/src/redisplay.c Sat Feb 20 03:46:22 2010 -0600 @@ -728,7 +728,7 @@ struct display_block *dbp = Dynarr_atp (dl->display_blocks, elt); /* "add" the block to the list */ - Dynarr_increment (dl->display_blocks); + Dynarr_incrementr (dl->display_blocks); /* initialize and return */ dbp->type = type; @@ -1221,7 +1221,7 @@ if (local) Dynarr_add (data->db->runes, *crb); else - Dynarr_increment (data->db->runes); + Dynarr_incrementr (data->db->runes); data->pixpos += width; @@ -4532,7 +4532,7 @@ if (Dynarr_length (dla) == 0) { if (Dynarr_largest (dla) > 0) - Dynarr_increment (dla); + Dynarr_incrementr (dla); else { struct display_line modeline; @@ -5301,9 +5301,10 @@ /* -Info on Re-entrancy crashes, with backtraces given: - - (Info-goto-node "(internals)Nasty Bugs due to Reentrancy in Redisplay Structures handling QUIT") +Info on reentrancy crashes, with backtraces given: + + (Info-goto-node "(internals)Critical Redisplay Sections") + */ @@ -5405,7 +5406,7 @@ if (pos_of_dlp < 0) Dynarr_add (dla, *dlp); else if (pos_of_dlp == Dynarr_length (dla)) - Dynarr_increment (dla); + Dynarr_incrementr (dla); else ABORT (); @@ -5459,13 +5460,13 @@ if (!in_display) depth = enter_redisplay_critical_section (); - /* This is one spot where a re-entrancy crash will occur, due to a check + /* This is one spot where a reentrancy crash will occur, due to a check in the dynarr to make sure it isn't "locked" */ /* -Info on Re-entrancy crashes, with backtraces given: - - (Info-goto-node "(internals)Nasty Bugs due to Reentrancy in Redisplay Structures handling QUIT") +Info on reentrancy crashes, with backtraces given: + + (Info-goto-node "(internals)Critical Redisplay Sections") */ Dynarr_reset (dla); @@ -5628,7 +5629,7 @@ if (pos_of_dlp < 0) Dynarr_add (dla, *dlp); else if (pos_of_dlp == Dynarr_length (dla)) - Dynarr_increment (dla); + Dynarr_incrementr (dla); else ABORT (); @@ -8469,7 +8470,7 @@ return; } - Dynarr_insert_many_at_start (cache, Dynarr_begin (internal_cache), + Dynarr_prepend_many (cache, Dynarr_begin (internal_cache), ic_elt + 1); }