GNU bug report logs - #75828
[PATCH] Correct make-frame docstring vis-a-vis make_frame.c

Previous Next

Package: emacs;

Reported by: Ship Mints <shipmints <at> gmail.com>

Date: Sat, 25 Jan 2025 14:15:02 UTC

Severity: minor

Tags: patch

Fixed in version 31.1

Done: Stefan Kangas <stefankangas <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: martin rudalics <rudalics <at> gmx.at>
To: Ship Mints <shipmints <at> gmail.com>
Cc: 75828 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>, stefankangas <at> gmail.com
Subject: bug#75828: [PATCH] Correct make-frame docstring vis-a-vis make_frame.c
Date: Mon, 27 Jan 2025 19:00:55 +0100
[Message part 1 (text/plain, inline)]
> These are the two options on the table?
>
> (let ((frame-inhibit-make-frame-hidden-buffer-check t))
>    (with-current-buffer " *string-pixel-width*"
>      (make-frame)))
> ;; could also do this if users want the behavior all the time
> (setq frame-inhibit-make-frame-hidden-buffer-check t)
>
> (with-current-buffer " *string-pixel-width*"
>    (setq-local frame-inhibit-make-frame-hidden-buffer-check t)
>    (make-frame)
>    (kill-local-variable 'frame-inhibit-make-frame-hidden-buffer-check))
>
> I prefer the former over the latter.

I prefer these ones

(let ((frame-inhibit-make-frame-hidden-buffer-check t))
  (with-current-buffer " *string-pixel-width*"
    (make-frame)))

;; Set the default for all buffers.
(setq-default frame-inhibit-make-frame-hidden-buffer-check t)

;; Set the buffer-local value.
(with-current-buffer " *string-pixel-width*"
  (setq frame-inhibit-make-frame-hidden-buffer-check t))

with the attached patch.

> It seems harmonious with other
> frame-related controls such as frame-resize-pixelwise
> and frame-inhibit-implied-resize, unless I'm missing some subtlety, which I
> surely am.

The subtlety is that a leading space in a buffer name is a buffer-local
property, a property which affects only the buffer wearing that name and
no other buffers.

'frame-resize-pixelwise' and 'frame-inhibit-implied-resize' are (or
better should be) frame properties, properties that affect a frame and
consequently all buffers displayed on that frame.

martin
[frame.c.diff (text/x-patch, attachment)]

This bug report was last modified 96 days ago.

Previous Next


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