GNU bug report logs - #76399
[PATCH] Change 'fullscreen' references to 'maximized'

Previous Next

Full log

Message #8 received at 76399 <at> debbugs.gnu.org (full text, mbox):

From: Ship Mints <shipmints <at> gmail.com>
To: 76399 <at> debbugs.gnu.org
Subject: Re: bug#76399: [PATCH] Change 'fullscreen' references to 'maximized'
Date: Tue, 18 Feb 2025 14:37:38 -0500

[Message part 1 (text/plain, inline)]

I did just find two references to 'fullscreen used as an alias to
'fullboth, but it's not documented and not implemented in all front ends
so its use is not portable.  I suggest we remove these or we document it
and correct all the front ends.  It looks like copy pasta, rather than by
intent.

pgtkfns.c:

static Lisp_Object
frame_geometry (Lisp_Object frame, Lisp_Object attribute)
{
  struct frame *f = decode_live_frame (frame);
  Lisp_Object fullscreen_symbol = Fframe_parameter (frame, Qfullscreen);
  bool fullscreen = (EQ (fullscreen_symbol, Qfullboth)
    || EQ (fullscreen_symbol, Qfullscreen));

nsfns.m:

static Lisp_Object
frame_geometry (Lisp_Object frame, Lisp_Object attribute)
{
  struct frame *f = decode_live_frame (frame);
  Lisp_Object fullscreen_symbol = Fframe_parameter (frame, Qfullscreen);
  bool fullscreen = (EQ (fullscreen_symbol, Qfullboth)
    || EQ (fullscreen_symbol, Qfullscreen));

On Tue, Feb 18, 2025 at 12:50 PM Ship Mints <shipmints <at> gmail.com> wrote:

> In frame.el's toggle-frame-fullscreen and toggle-frame-maximized, these
> look like either typos, or vestigial references.  The frame-parameter
> fullscreen is never set to 'fullscreen nor is the value 'fullscreen used in
> other places in Emacs.  It is an undocumented value, appearing nowhere.
> Unless I missed something.
>
> I reviewed below sources and relevant headers:
>
> android.c
> androidfns.c
> androidterm.c
> emacs.c
> frame.c
> gtkutil.c
> haikufns.c
> haikuterm.c
> pgtkfns.c
> pgtkterm.c
> term.c
> w32fns.c
> w32term.c
> xfns.c
> xmenu.c
> xterm.c
> nsfns.m
> nsterm.m
>
> -Stephane
>

[Message part 2 (text/html, inline)]

Previous Next

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