From: Miles Bader <miles@gnu.org>
Subject: Proposed patches
Date: 25 Feb 2004 08:00:28 +0900 [thread overview]
Message-ID: <87vflwhylv.fsf@tc-1-100.kawasaki.gol.ne.jp> (raw)
[-- Attachment #1: Type: text/plain, Size: 386 bytes --]
Hi,
The following patches are from my emacs tiling branch, and fix a number
of annoying crash bugs, but also seem to apply equally well to CVS HEAD
(however, at least in the case of the third patch, I couldn't actually
reproduce the crash using CVS HEAD).
Do these changes look alright to people? [If I don't hear any
complaints, I'll apply them to CVS]
Thanks,
-Miles
patch-1:
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch-1 --]
[-- Type: text/x-patch, Size: 639 bytes --]
2004-02-10 Miles Bader <miles@gnu.org>
* xfns.c (lookup_image): Remove xassert(!interrupt_input_blocked);
BLOCK_INPUT can be nested, so it doesn't make much sense.
--- orig/src/xfns.c
+++ mod/src/xfns.c
@@ -1,5 +1,5 @@
/* Functions for the X window system.
- Copyright (C) 1989, 92, 93, 94, 95, 96, 97, 98, 99, 2000, 01, 02, 03
+ Copyright (C) 1989, 92, 93, 94, 95, 96, 97, 98, 99, 2000,01,02,03,04
Free Software Foundation.
This file is part of GNU Emacs.
@@ -5414,7 +5414,6 @@
}
UNBLOCK_INPUT;
- xassert (!interrupt_input_blocked);
}
/* We're using IMG, so set its timestamp to `now'. */
[-- Attachment #3: Type: text/plain, Size: 12 bytes --]
patch-2:
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #4: patch-2 --]
[-- Type: text/x-patch, Size: 699 bytes --]
2004-02-11 Miles Bader <miles@gnu.org>
* xdisp.c (produce_image_glyph): Force negative descents to zero.
--- orig/src/xdisp.c
+++ mod/src/xdisp.c
@@ -17827,6 +17827,11 @@
it->descent = it->phys_descent = img->height + 2 * img->vmargin - it->ascent;
it->pixel_width = img->width + 2 * img->hmargin;
+ /* It's quite possible for images to have an ascent greater than
+ their height, so don't get confused in that case. */
+ if (it->descent < 0)
+ it->descent = 0;
+
/* If this glyph is alone on the last line, adjust it.ascent to minimum row ascent. */
face_ascent = face->font ? FONT_BASE (face->font) : FRAME_BASELINE_OFFSET (it->f);
if (face_ascent > it->ascent)
[-- Attachment #5: Type: text/plain, Size: 12 bytes --]
patch-3:
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #6: patch-3 --]
[-- Type: text/x-patch, Size: 1088 bytes --]
2004-02-25 Miles Bader <miles@gnu.org>
* xdisp.c (check_it): Check string/string_pos consistency.
(init_iterator): Initialize string-related fields properly.
--- orig/src/xdisp.c
+++ mod/src/xdisp.c
@@ -1905,10 +1905,14 @@
xassert (STRINGP (it->string));
xassert (IT_STRING_CHARPOS (*it) >= 0);
}
- else if (it->method == next_element_from_buffer)
+ else
{
- /* Check that character and byte positions agree. */
- xassert (IT_CHARPOS (*it) == BYTE_TO_CHAR (IT_BYTEPOS (*it)));
+ xassert (IT_STRING_CHARPOS (*it) < 0);
+ if (it->method == next_element_from_buffer)
+ {
+ /* Check that character and byte positions agree. */
+ xassert (IT_CHARPOS (*it) == BYTE_TO_CHAR (IT_BYTEPOS (*it)));
+ }
}
if (it->dpvec)
@@ -2021,6 +2025,8 @@
it->current.overlay_string_index = -1;
it->current.dpvec_index = -1;
it->base_face_id = base_face_id;
+ it->string = Qnil;
+ IT_STRING_CHARPOS (*it) = IT_STRING_BYTEPOS (*it) = -1;
/* The window in which we iterate over current_buffer: */
XSETWINDOW (it->window, w);
[-- Attachment #7: Type: text/plain, Size: 27 bytes --]
--
Run away! Run away!
[-- Attachment #8: Type: text/plain, Size: 141 bytes --]
_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://mail.gnu.org/mailman/listinfo/emacs-devel
next reply other threads:[~2004-02-24 23:00 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-02-24 23:00 Miles Bader [this message]
2004-02-25 5:37 ` Proposed patches Eli Zaretskii
2004-02-25 6:39 ` Alexander Winston
2004-02-25 6:50 ` Miles Bader
2004-02-25 10:06 ` Eli Zaretskii
2004-02-25 14:18 ` Stefan Monnier
2004-02-25 14:29 ` David Kastrup
2004-02-25 12:09 ` Kim F. Storm
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87vflwhylv.fsf@tc-1-100.kawasaki.gol.ne.jp \
--to=miles@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.