Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Kevin Buettner <kevinb@redhat.com>
To: Guinevere Larsen <guinevere@redhat.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH v3 3/3] gdb: extend the [[N]]::foo syntax for files
Date: Wed, 9 Jul 2025 19:02:26 -0700	[thread overview]
Message-ID: <20250709190226.41d1be21@f41-zbm-amd> (raw)
In-Reply-To: <20250619180540.2315103-4-guinevere@redhat.com>

Hi Gwen,

On Thu, 19 Jun 2025 15:05:40 -0300
Guinevere Larsen <guinevere@redhat.com> wrote:

> diff --git a/gdb/parse.c b/gdb/parse.c
> index 64653c8ae6e..6edd9058dcb 100644
> --- a/gdb/parse.c
> +++ b/gdb/parse.c
> @@ -421,7 +421,8 @@ parse_exp_in_context (const char **stringptr,
> CORE_ADDR pc, 
>    parser_state ps (lang, get_current_arch (), expression_context_block,
>  		   expression_context_pc, flags, *stringptr,
> -		   completer != nullptr, tracker);
> +		   completer != nullptr, tracker,
> +		   current_program_space->solib_ops ());
>  
>    scoped_restore_current_language lang_saver (lang->la_language);
>  

I'm seeing a build error related to the bit above:

/mesquite2/sourceware-git/worktree-review/gdb/parse.c: In function
‘expression_up parse_exp_in_context(const char**, CORE_ADDR, const block*,
parser_flags, innermost_block_tracker*,
std::unique_ptr<expr_completion_base>*)’:
/mesquite2/sourceware-git/worktree-review/gdb/parse.c:425:53: error:
invalid conversion from ‘const solib_ops*’ to ‘solib_ops*’ [-fpermissive]
425 |                    current_program_space->solib_ops ()); |
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~ |
                      | |
  const solib_ops* In file included from
/mesquite2/sourceware-git/worktree-review/gdb/parse.c:41:
/mesquite2/sourceware-git/worktree-review/gdb/parser-defs.h:152:28: note:
initializing argument 9 of ‘parser_state::parser_state(const
language_defn*, gdbarch*, const block*, CORE_ADDR, parser_flags, const
char*, bool, innermost_block_tracker*, solib_ops*)’ 152 |
solib_ops *ops) |                 ~~~~~~~~~~~^~~

Kevin


  reply	other threads:[~2025-07-10  2:03 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-19 18:05 [PATCH v3 0/3] Introduce syntax for linker-namespace specific symbols Guinevere Larsen
2025-06-19 18:05 ` [PATCH v3 1/3] gdb: make lookup_minimal_symbol_linkage work with linker namespaces Guinevere Larsen
2025-06-19 18:05 ` [PATCH v3 2/3] gdb: Make the parser recognize the [[N]] syntax for variables Guinevere Larsen
2025-06-20  6:57   ` Eli Zaretskii
2025-07-07 19:18     ` Guinevere Larsen
2025-06-19 18:05 ` [PATCH v3 3/3] gdb: extend the [[N]]::foo syntax for files Guinevere Larsen
2025-07-10  2:02   ` Kevin Buettner [this message]
2025-07-10 14:47     ` Guinevere Larsen
2025-07-10  2:27   ` Kevin Buettner
2025-07-10 15:07     ` Guinevere Larsen
2025-07-10 21:56       ` Kevin Buettner
2025-07-07 19:01 ` [PING][PATCH v3 0/3] Introduce syntax for linker-namespace specific symbols Guinevere Larsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250709190226.41d1be21@f41-zbm-amd \
    --to=kevinb@redhat.com \
    --cc=gdb-patches@sourceware.org \
    --cc=guinevere@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox