Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom Tromey <tom@tromey.com>
To: Will Hawkins <hawkinsw@obs.cr>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH v2] Make source.c:search_command_helper use source cache
Date: Fri, 19 Apr 2024 13:40:13 -0600	[thread overview]
Message-ID: <8734rhuoo2.fsf@tromey.com> (raw)
In-Reply-To: <20240325140103.157217-1-hawkinsw@obs.cr> (Will Hawkins's message of "Mon, 25 Mar 2024 10:00:52 -0400")

>>>>> "Will" == Will Hawkins <hawkinsw@obs.cr> writes:

Will> The current implementation of search_command_helper accesses the line
Will> offsets of the current program spaces's source code through
Will> the source cache but then accesses its contents through the disk. This
Will> PR updates the implementation so that the access of the contents is also
Will> through the source cache.

It seems to me that this will result in some changes, because the source
cache may hold text that has had styling applied, and so may have
embedded ANSI escape sequences -- which the search command isn't
expecting.

If this is needed for your source-embedding work then perhaps the source
cache should store an unmodified copy of the source as well, at least in
the case that styles have been applied.

Tom

  parent reply	other threads:[~2024-04-19 19:40 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-24  1:18 [PATCH] " Will Hawkins
2024-03-25 14:00 ` [PATCH v2] " Will Hawkins
2024-03-29 14:46   ` Will Hawkins
2024-04-08 18:12     ` Will Hawkins
2024-04-18  1:28       ` Will Hawkins
2024-04-19 19:41     ` Tom Tromey
2024-04-19 22:28       ` Will Hawkins
2024-04-19 19:40   ` Tom Tromey [this message]
2024-04-19 22:30     ` Will Hawkins

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=8734rhuoo2.fsf@tromey.com \
    --to=tom@tromey.com \
    --cc=gdb-patches@sourceware.org \
    --cc=hawkinsw@obs.cr \
    /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