GNU bug report logs - #22072
Error in display-startup-screen if create-image fails

Previous Next

Package: emacs;

Reported by: Richard Copley <rcopley <at> gmail.com>

Date: Wed, 2 Dec 2015 01:55:02 UTC

Severity: normal

Tags: fixed

Fixed in version 27.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Richard Copley <rcopley <at> gmail.com>
To: Stephen Leake <stephen_leake <at> stephe-leake.org>
Cc: 22072 <at> debbugs.gnu.org
Subject: bug#22072: Error in display-startup-screen if create-image fails
Date: Wed, 2 Dec 2015 20:07:01 +0000
Thanks Stephen. More haste, less speed ... you shouldn't have changed
"let*" to "let" and the parentheses don't match. After fixing those
things, yes, doing as you suggest allows my broken Emacs to start
without the fancy image and without error.

On 2 December 2015 at 12:36, Stephen Leake
<stephen_leake <at> stephe-leake.org> wrote:
> Richard Copley <rcopley <at> gmail.com> writes:
>
> Can you test this patch:
>
> diff --git a/lisp/startup.el b/lisp/startup.el
> index 1346310..a281f86 100644
> --- a/lisp/startup.el
> +++ b/lisp/startup.el
> @@ -1874,13 +1874,13 @@ we put it on this frame."
>                   (image-type-available-p 'pbm)))
>      (let ((frame (fancy-splash-frame)))
>        (when frame
> -       (let* ((img (create-image (fancy-splash-image-file)))
> -              (image-height (and img (cdr (image-size img nil frame))))
> -              ;; We test frame-height so that, if the frame is split
> -              ;; by displaying a warning, that doesn't cause the normal
> -              ;; splash screen to be used.
> -              (frame-height (1- (frame-height frame))))
> -         (> frame-height (+ image-height 19)))))))
> +       (let ((img (create-image (fancy-splash-image-file))))
> +              (image-height (and img (cdr (image-size img nil frame))))
> +              ;; We test frame-height so that, if the frame is split
> +              ;; by displaying a warning, that doesn't cause the normal
> +              ;; splash screen to be used.
> +              (frame-height (and img (1- (frame-height frame)))))
> +         (and img (> frame-height (+ image-height 19))))))))
>
> --
> -- Stephe




This bug report was last modified 5 years and 239 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.