* [PATCH] Printing of large objects not interruptable
@ 2014-12-08 22:39 Alastair Robertson
2014-12-20 17:18 ` Joel Brobecker
0 siblings, 1 reply; 2+ messages in thread
From: Alastair Robertson @ 2014-12-08 22:39 UTC (permalink / raw)
To: gdb-patches
This patch makes the counting of repeated characters interruptable,
relating to bug 17613
https://sourceware.org/bugzilla/show_bug.cgi?id=17613
This is my first patch, so sorry if I forgot to include something or
didn't format it correctly.
2014-12-07 Alastair Robertson <alastair@ajor.co.uk>
PR gdb/17613
* valprint.c (count_next_character): Call QUIT
diff --git a/gdb/valprint.c b/gdb/valprint.c
index b995304..8d743b1 100644
--- a/gdb/valprint.c
+++ b/gdb/valprint.c
@@ -2151,6 +2151,8 @@ count_next_character (struct wchar_iterator *iter,
while (1)
{
+ QUIT;
+
/* Get the next character. */
d.num_chars
= wchar_iterate (iter, &d.result, &chars, &d.buf, &d.buflen);
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Printing of large objects not interruptable
2014-12-08 22:39 [PATCH] Printing of large objects not interruptable Alastair Robertson
@ 2014-12-20 17:18 ` Joel Brobecker
0 siblings, 0 replies; 2+ messages in thread
From: Joel Brobecker @ 2014-12-20 17:18 UTC (permalink / raw)
To: Alastair Robertson; +Cc: gdb-patches
> This patch makes the counting of repeated characters interruptable,
> relating to bug 17613
> https://sourceware.org/bugzilla/show_bug.cgi?id=17613
>
> This is my first patch, so sorry if I forgot to include something or
> didn't format it correctly.
>
> 2014-12-07 Alastair Robertson <alastair@ajor.co.uk>
>
> PR gdb/17613
> * valprint.c (count_next_character): Call QUIT
This patch looks good to me.
If you could add a short description of what this PR is about in
the commit's revision log, so as to avoid the need to go through
a PR, which is not always a straightforward (or even concise read),
that would be very good.
In terms of items for submission, we have a checklist:
https://sourceware.org/gdb/wiki/ContributionChecklist
There is a lot of important information there, if you haven't read
it yet.
As far as this patch is concerned, the only bits that are missing
are a period at the end of your sentence in your ChangeLog entry;
and an indication that you ran the testsuite before and after
your patch, showing no regression (this one is unlikely to produce
any, but better safe than sorry) - that indication should be in
the commit's revision log, including the platform that was used
to perform the testing.
Please re-send an updated patch, and I will push it for you.
Thank you!
> diff --git a/gdb/valprint.c b/gdb/valprint.c
> index b995304..8d743b1 100644
> --- a/gdb/valprint.c
> +++ b/gdb/valprint.c
> @@ -2151,6 +2151,8 @@ count_next_character (struct wchar_iterator *iter,
>
> while (1)
> {
> + QUIT;
> +
> /* Get the next character. */
> d.num_chars
> = wchar_iterate (iter, &d.result, &chars, &d.buf, &d.buflen);
--
Joel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-12-20 17:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-08 22:39 [PATCH] Printing of large objects not interruptable Alastair Robertson
2014-12-20 17:18 ` Joel Brobecker
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).