bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Gerd Möllmann]

On macOS 12.4, selecting Options -> Set Default Font from the menu bar 
reproducibly prints a message to the terminal:

emacs[76309:7042282] It's not legal to call -layoutSubtreeIfNeeded on
a view which is already being laid out.  If you are implementing the
view's -layout method, you can call -[super layout] instead. Break on
void _NSDetectedLayoutRecursion(void) to debug.  This will be logged
only once.  This may break in the future.

Hopefully nobody get's sued because of that illegal action...

Backtrace from lldb:

 * frame #0: 0x000000019e2f5f40 AppKit`_NSDetectedLayoutRecursion
    frame #1: 0x000000019dac4148 AppKit`-[NSWindow(NSConstraintBasedLayout) _layoutViewTree] + 148
    frame #2: 0x000000019dac11d0 AppKit`-[NSWindow _oldPlaceWindow:fromServer:] + 584
    frame #3: 0x000000019dac0050 AppKit`-[NSWindow _setFrameCommon:display:fromServer:] + 2220
    frame #4: 0x000000019db854cc AppKit`-[NSThemeFrame _growWindowReshapeContentAndToolbarView:withOldToolbarFrameSize:animate:] + 1232
    frame #5: 0x000000019db84ddc AppKit`-[NSThemeFrame _reshapeContentAndToolbarView:withOldToolbarFrameSize:resizeWindow:animate:] + 300
    frame #6: 0x000000019db738bc AppKit`-[NSThemeFrame _toolbarFrameSizeChanged:oldSize:] + 76
    frame #7: 0x000000019db737f4 AppKit`-[NSWindow _toolbarFrameSizeChanged:oldSize:] + 96
    frame #8: 0x000000019db68870 AppKit`-[NSToolbarView _layoutDirtyItemViewersAndTileToolbar] + 428
    frame #9: 0x000000019db8b114 AppKit`-[NSToolbarView layout] + 88
    frame #10: 0x000000019dac5700 AppKit`_NSViewLayout + 688
    frame #11: 0x000000019dac5178 AppKit`-[NSView _layoutSubtreeWithOldSize:] + 380
    frame #12: 0x000000019dac52e4 AppKit`-[NSView _layoutSubtreeWithOldSize:] + 744
    frame #13: 0x000000019dac52e4 AppKit`-[NSView _layoutSubtreeWithOldSize:] + 744
    frame #14: 0x000000019dac52e4 AppKit`-[NSView _layoutSubtreeWithOldSize:] + 744
    frame #15: 0x000000019dac4600 AppKit`-[NSView _layoutSubtreeIfNeededAndAllowTemporaryEngine:] + 996
    frame #16: 0x000000019dac4148 AppKit`-[NSWindow(NSConstraintBasedLayout) _layoutViewTree] + 148
    frame #17: 0x000000019db3b8ec AppKit`-[NSWindow(NSConstraintBasedLayout) layoutIfNeeded] + 308
    frame #18: 0x000000019dba7388 AppKit`-[NSWindow _setUpFirstResponderBeforeBecomingVisible] + 84
    frame #19: 0x000000019dba6750 AppKit`-[NSWindow _doWindowWillBeVisibleAsSheet:] + 164
    frame #20: 0x000000019dba507c AppKit`-[NSWindow _reallyDoOrderWindowAboveOrBelow:relativeTo:findKey:forCounter:force:isModal:] + 1232
    frame #21: 0x000000019dba4790 AppKit`-[NSWindow _reallyDoOrderWindow:relativeTo:findKey:forCounter:force:isModal:] + 132
    frame #22: 0x000000019dba377c AppKit`-[NSWindow _doOrderWindow:relativeTo:findKey:forCounter:force:isModal:] + 324
    frame #23: 0x000000010088d424 emacs`-[EmacsView showFontPanel](self=0x0000000108431400, _cmd="showFontPanel") at nsterm.m:6585:3
    frame #24: 0x000000010090d670 emacs`Fx_select_font(frame=0x0000000000000000, ignored=0x0000000000000000) at nsfns.m:1662:22

A related bug might be bug#53481 (status done, fixed by Alan Third).

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Gerd Möllmann <gerd.moellmann@gmail.com> writes:

> On macOS 12.4, selecting Options -> Set Default Font from the menu bar 
> reproducibly prints a message to the terminal:
>
> emacs[76309:7042282] It's not legal to call -layoutSubtreeIfNeeded on
> a view which is already being laid out.  If you are implementing the
> view's -layout method, you can call -[super layout] instead. Break on
> void _NSDetectedLayoutRecursion(void) to debug.  This will be logged
> only once.  This may break in the future.
>

I wonder if this is a bug in macOS 12.4 (concretely, in NSFontManager's
orderFrontFontPanel) and not in Emacs.  For example, I can reproduce the
same warning if I try to open the Fonts panel in the standard TextEdit
app created by Apple.

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Eli Zaretskii]

> Cc: 56259@debbugs.gnu.org
> Date: Mon, 27 Jun 2022 17:05:16 +0200
> From:  Daniel Martín via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
> 
> Gerd Möllmann <gerd.moellmann@gmail.com> writes:
> 
> > On macOS 12.4, selecting Options -> Set Default Font from the menu bar 
> > reproducibly prints a message to the terminal:
> >
> > emacs[76309:7042282] It's not legal to call -layoutSubtreeIfNeeded on
> > a view which is already being laid out.  If you are implementing the
> > view's -layout method, you can call -[super layout] instead. Break on
> > void _NSDetectedLayoutRecursion(void) to debug.  This will be logged
> > only once.  This may break in the future.
> >
> 
> I wonder if this is a bug in macOS 12.4 (concretely, in NSFontManager's
> orderFrontFontPanel) and not in Emacs.  For example, I can reproduce the
> same warning if I try to open the Fonts panel in the standard TextEdit
> app created by Apple.

Then let's sue Apple for its illegal calls.

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Richard Stallman]

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

  > > I wonder if this is a bug in macOS 12.4 (concretely, in NSFontManager's
  > > orderFrontFontPanel) and not in Emacs.  For example, I can reproduce the
  > > same warning if I try to open the Fonts panel in the standard TextEdit
  > > app created by Apple.

  > Then let's sue Apple for its illegal calls.

None of these things is "illegal" -- that's the wrong word.
I'm not sure what people mean by using it in this context,
but I don't think there is a law against it.

Apple surely has no legal obligation to implement the feature we
would like to use, so there is no basis for a lawsuit against Apple.

This is why we urge people not to use the word "illegal" to say
that some way of using a program is considered invalid or not supported.
See the node GNU Manuals in the GNU Coding Standards.

-- 
Dr Richard Stallman (https://stallman.org)
Chief GNUisance of the GNU Project (https://gnu.org)
Founder, Free Software Foundation (https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Eli Zaretskii]

> From: Richard Stallman <rms@gnu.org>
> Cc: mardani29@yahoo.es, gerd.moellmann@gmail.com,
> 	56259@debbugs.gnu.org
> Date: Wed, 29 Jun 2022 23:09:10 -0400
> 
>   > > I wonder if this is a bug in macOS 12.4 (concretely, in NSFontManager's
>   > > orderFrontFontPanel) and not in Emacs.  For example, I can reproduce the
>   > > same warning if I try to open the Fonts panel in the standard TextEdit
>   > > app created by Apple.
> 
>   > Then let's sue Apple for its illegal calls.
> 
> None of these things is "illegal" -- that's the wrong word.

There's nothing illegal in joking about that, either.

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Gerd Möllmann]

[-- Attachment #1: Type: text/plain, Size: 297 bytes --]



> On 2022-06-30,, at 7:20 , Eli Zaretskii <eliz@gnu.org> wrote:
> 
>>> Then let's sue Apple for its illegal calls.
>> 
>> None of these things is "illegal" -- that's the wrong word.
> 
> There's nothing illegal in joking about that, either.

And there could be a lot of money in it for the FSF.

[-- Attachment #2: Message signed with OpenPGP --]
[-- Type: application/pgp-signature, Size: 874 bytes --]

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Richard Stallman]

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

  > > None of these things is "illegal" -- that's the wrong word.

  > There's nothing illegal in joking about that, either.

That joking is not illegal, but it can be confusing.
(I couldn't tell it was a joke.)

-- 
Dr Richard Stallman (https://stallman.org)
Chief GNUisance of the GNU Project (https://gnu.org)
Founder, Free Software Foundation (https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Gerd Möllmann <gerd.moellmann@gmail.com> writes:

> On macOS 12.4, selecting Options -> Set Default Font from the menu bar 
> reproducibly prints a message to the terminal:
>
> emacs[76309:7042282] It's not legal to call -layoutSubtreeIfNeeded on
> a view which is already being laid out.  If you are implementing the
> view's -layout method, you can call -[super layout] instead. Break on
> void _NSDetectedLayoutRecursion(void) to debug.  This will be logged
> only once.  This may break in the future.
>
> Hopefully nobody get's sued because of that illegal action...

I get the feeling this is one of Apple's bugs, since it happens to every
program's fonts panel, including those that come with Mac OS.

So let's sue Apple!

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Gerd Möllmann]

[-- Attachment #1.1: Type: text/plain, Size: 282 bytes --]



> On 2022-06-28,, at 3:24 , Po Lu <luangruo@yahoo.com> wrote:
> 
> So let's sue Apple!

We should first try in good faith, that always makes a good impression in court.

We could assign the bug to Tim Cook and ask him to solve it. With a deadline for the fix, of course.

[-- Attachment #1.2: Type: text/html, Size: 1182 bytes --]

[-- Attachment #2: Message signed with OpenPGP --]
[-- Type: application/pgp-signature, Size: 874 bytes --]

bug#56259: 29.0.50; Illegal -layoutSubtreeIfNeeded on NS

[Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors]

tags 56259 notabug
close 56259
quit

I think the general consensus was that this message is a bug in some
Apple framework and not in Emacs, so I'm closing the bug report.

bug#58540: 29.0.50; While opening the font dialog

[Stefan Kangas]

Daniel Martín <mardani29@yahoo.es> writes:

> tags 56259 notabug
> close 56259
> quit
>
> I think the general consensus was that this message is a bug in some
> Apple framework and not in Emacs, so I'm closing the bug report.

The bug was left open, so I'm closing it now.

bug#58540: 29.0.50; While opening the font dialog

[Pedro Andres Aranda Gutierrez]

[-- Attachment #1: Type: text/plain, Size: 3610 bytes --]

After emacs -Q

Open font dialog

I got the following message on thee console

2022-10-15 08:41:32.097 Emacs[1840:44988] It's not legal to call
-layoutSubtreeIfNeeded on a view which is already being laid out.  If you
are implementing the view's -layout method, you can call -[super layout]
instead. Break on void _NSDetectedLayoutRecursion(void) to debug.  This
will be logged only once.  This may break in the future.


In GNU Emacs 29.0.50 (build 1, x86_64-apple-darwin21.6.0, NS
 appkit-2113.60 Version 12.6 (Build 21G115)) of 2022-10-14 built on
 paag-MacBookAir
Repository revision: b175a9fb7eea49cdf8171821498f2c5efaaecd98
Repository branch: master
Windowing system distributor 'Apple', version 10.3.2113
System Description:  macOS 12.6

Configured using:
 'configure PKG_CONFIG_PATH=/usr/local/lib/pkgconfig
 LDFLAGS=-L/usr/local/lib CPPFLAGS=-I/usr/local/include CC=clang
 OBJC=clang 'CFLAGS=-g -O3' LIBXML2_CFLAGS=-I12.3/usr/include/libxml2
 LIBXML2_LIBS=-lxml2'

Configured features:
ACL GMP GNUTLS JPEG JSON LIBXML2 MODULES NOTIFY KQUEUE NS PDUMPER PNG
SQLITE3 THREADS TIFF TOOLKIT_SCROLL_BARS WEBP ZLIB

Important settings:
  value of $LC_ALL: es_ES.UTF-8
  value of $LANG: es_ES.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  show-paren-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  line-number-mode: t
  indent-tabs-mode: t
  transient-mark-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message mailcap yank-media puny dired
dired-loaddefs rfc822 mml mml-sec password-cache epa derived epg rfc6068
epg-config gnus-util text-property-search time-date subr-x mm-decode
mm-bodies mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader
cl-loaddefs cl-lib sendmail rfc2047 rfc2045 ietf-drums mm-util
mail-prsvr mail-utils rmc iso-transl tooltip eldoc paren electric
uniquify ediff-hook vc-hooks lisp-float-type elisp-mode mwheel
term/ns-win ns-win ucs-normalize mule-util term/common-win tool-bar dnd
fontset image regexp-opt fringe tabulated-list replace newcomment
text-mode lisp-mode prog-mode register page tab-bar menu-bar rfn-eshadow
isearch easymenu timer select scroll-bar mouse jit-lock font-lock syntax
font-core term/tty-colors frame minibuffer nadvice seq simple cl-generic
indonesian philippine cham georgian utf-8-lang misc-lang vietnamese
tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek
romanian slovak czech european ethiopic indian cyrillic chinese
composite emoji-zwj charscript charprop case-table epa-hook
jka-cmpr-hook help abbrev obarray oclosure cl-preloaded button loaddefs
faces cus-face macroexp files window text-properties overlay sha1 md5
base64 format env code-pages mule custom widget keymap
hashtable-print-readable backquote threads kqueue cocoa ns multi-tty
make-network-process emacs)

Memory information:
((conses 16 37761 8788)
 (symbols 48 5010 0)
 (strings 32 13067 1813)
 (string-bytes 1 371585)
 (vectors 16 10339)
 (vector-slots 8 161411 14552)
 (floats 8 25 20)
 (intervals 56 275 0)
 (buffers 1000 11))

-- 
Fragen sind nicht da um beantwortet zu werden,
Fragen sind da um gestellt zu werden
Georg Kreisler

Headaches with a Juju log:
unit-basic-16: 09:17:36 WARNING juju.worker.uniter.operation we should run
a leader-deposed hook here, but we can't yet

[-- Attachment #2: Type: text/html, Size: 4202 bytes --]

bug#58540: 29.0.50; While opening the font dialog

[Eli Zaretskii]

> From: Pedro Andres Aranda Gutierrez <paaguti@gmail.com>
> Date: Sat, 15 Oct 2022 08:44:28 +0200
> 
> After emacs -Q
> 
> Open font dialog

How do you open that dialog, exactly?

> I got the following message on thee console
> 
> 2022-10-15 08:41:32.097 Emacs[1840:44988] It's not legal to call -layoutSubtreeIfNeeded on a view which is
> already being laid out.  If you are implementing the view's -layout method, you can call -[super layout]
> instead. Break on void _NSDetectedLayoutRecursion(void) to debug.  This will be logged only once.  This
> may break in the future.

This is macOS specific, because I don't see anything like that here.

Thanks.

bug#58540: 29.0.50; While opening the font dialog

[Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Eli Zaretskii <eliz@gnu.org> writes:

> This is macOS specific, because I don't see anything like that here.

Not only that, but it also shows up on every other Mac OS app when
opening the font dialog.  So let's sue Apple for their illegal
activities.  (Doesn't this joke ring a bell? I think this bug report is
a duplicate of another, but I can't find it.)

bug#58540: 29.0.50; While opening the font dialog

[Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Po Lu via "Bug reports for GNU Emacs, the Swiss army knife of text
editors" <bug-gnu-emacs@gnu.org> writes:

> Eli Zaretskii <eliz@gnu.org> writes:
>
>> This is macOS specific, because I don't see anything like that here.
>
> Not only that, but it also shows up on every other Mac OS app when
> opening the font dialog.  So let's sue Apple for their illegal
> activities.  (Doesn't this joke ring a bell? I think this bug report is
> a duplicate of another, but I can't find it.)

Yes, it's a duplicate of bug#56259.

bug#58540: 29.0.50; While opening the font dialog

[Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors]

forcemerge 58540 56259
thanks

Daniel Martín <mardani29@yahoo.es> writes:

> Yes, it's a duplicate of bug#56259.

Thanks, I've merged the two.