comparison src/insdel.h @ 5879:77d7b77909c2

Move extents.c to working in byte positions only; fix a bug, extent_detach() src/ChangeLog addition: 2015-03-27 Aidan Kehoe <kehoea@parhasard.net> Fix a small bug, extent_detach(); minimise needless char-byte conversion, extents.c, sticking to byte positions in general in this file. * extents.c: * extents.c (signal_single_extent_changed): Pass byte endpoints to gutter_extent_signal_changed_region_maybe(), buffer_extent_signal_changed_region(). * extents.c (extent_detach): Call signal_extent_changed() correctly, pass both extent endpoints rather than just the byte and character variants of the start. * extents.c (struct report_extent_modification_closure): Do this in terms of byte positions. * extents.c (report_extent_modification_mapper): Use byte positions, only converting to characters when we are definitely calling Lisp. * extents.c (report_extent_modification): Use byte positions in this API, move the byte-char conversion to our callers, simplifying extents.c (it all now works in byte positions). * extents.h: Update report_extent_modification's prototype. * gutter.c (gutter_extent_signal_changed_region_maybe): Use byte positions here, avoids needless byte-char conversion. * gutter.h: Update the prototype here. * insdel.c: * insdel.c (buffer_extent_signal_changed_region): Implement this in terms of byte positions. * insdel.c (signal_before_change): * insdel.c (signal_after_change): Call report_extent_modification() with byte positions, doing the char->byte conversion here rather than leaving it to extents.c. * insdel.h: * insdel.h (struct each_buffer_change_data): The extent unchanged info now describes bytecounts.
author Aidan Kehoe <kehoea@parhasard.net>
date Fri, 27 Mar 2015 23:39:49 +0000
parents 65d65b52d608
children
comparison
equal deleted inserted replaced
5878:916187a03b82 5879:77d7b77909c2
97 { 97 {
98 Charcount begin_unchanged, end_unchanged; 98 Charcount begin_unchanged, end_unchanged;
99 /* redisplay needs to know if a newline was deleted so its 99 /* redisplay needs to know if a newline was deleted so its
100 incremental-redisplay algorithm will fail */ 100 incremental-redisplay algorithm will fail */
101 int newline_was_deleted; 101 int newline_was_deleted;
102 Charcount begin_extent_unchanged, end_extent_unchanged; 102 Bytecount begin_extent_unchanged, end_extent_unchanged;
103 }; 103 };
104 104
105 /* Number of characters at the beginning and end of the buffer that 105 /* Number of characters at the beginning and end of the buffer that
106 have not changed since the last call to buffer_reset_changes(). 106 have not changed since the last call to buffer_reset_changes().
107 If no changes have occurred since then, both values will be -1. 107 If no changes have occurred since then, both values will be -1.
109 "Changed" means that the text has changed. */ 109 "Changed" means that the text has changed. */
110 110
111 #define BUF_BEGIN_UNCHANGED(buf) ((buf)->changes->begin_unchanged) 111 #define BUF_BEGIN_UNCHANGED(buf) ((buf)->changes->begin_unchanged)
112 #define BUF_END_UNCHANGED(buf) ((buf)->changes->end_unchanged) 112 #define BUF_END_UNCHANGED(buf) ((buf)->changes->end_unchanged)
113 113
114 /* Number of characters at the beginning and end of the buffer that 114 /* Number of bytes at the beginning and end of the buffer that
115 have not had a covering extent change since the last call to 115 have not had a covering extent change since the last call to
116 buffer_reset_changes (). If no changes have occurred since then, 116 buffer_reset_changes (). If no changes have occurred since then,
117 both values will be -1. 117 both values will be -1.
118 118
119 "Changed" means that the extents covering the text have changed. */ 119 "Changed" means that the extents covering the text have changed. */