Subject: Re: bug#77125: [PATCH] igc ; * etc/DEBUG: Fix gdb examples to: call ... (global_igc->arena)
From: Eli Zaretskii <[email-masked]>
To: Jeremy Bryant <[email-masked]>
Cc: [email-masked], [email-masked], [email-masked], [email-masked]
Date: Mon, 07 Apr 2025 14:18:45 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

> From: Jeremy Bryant <[email-masked]>
> Cc: [email-masked],  [email-masked], Helmut Eller
>  <[email-masked]>, Gerd Möllmann
>  <[email-masked]>
> Date: Sun, 06 Apr 2025 21:26:01 +0100
> 
> Eli Zaretskii <[email-masked]> writes:
> 
> >> From: Jeremy Bryant <[email-masked]>
> >>  and then release it afterwards with:
> >>  
> >> -     (gdb) call mps_arena_release(arena)
> >> +     (gdb) call mps_arena_release(global_igc->arena)
> >
> > Is it really true that the arena pointer is always global_igc->arena
> > in Emacs?  If so, I would suggest to tell that separately, because the
> > GDB calls to functions can have arguments constructed in many
> > different ways.  Something like
> >
> >   In Emacs, the arena is usually stored in 'global_igc->arena'.
> >
> > and leave the examples in their current general form.
> 
> OK, on that basis please find attached patch adding that line, with 'usually'.

Thanks, installed on the branch.

Please in the future remember to mention the bug number (when known)
in the commit log message, and also end single-line log messages with
a period.