[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Integrated: 8271356: Modify jdb to treat an empty command as a repeat of the previous command
From:       Jakob Cornell <duke () openjdk ! java ! net>
Date:       2021-10-29 17:53:17
Message-ID: JB91xuPmuJy_vqVe6nQC3odNnjIrN3wjXo8QOSquX1U=.5ec28585-d529-4d83-b255-23d6921e64f1 () github ! com
[Download RAW message or body]

On Sat, 28 Aug 2021 05:48:47 GMT, Jakob Cornell <duke@openjdk.java.net> wrote:

> This has been under discussion on and off for the past month or so on \
> serviceability-dev, and I think a CSR request is required, so this may be a work in \
> progress. 
> Notes on the patch:
> 
> - The `list` command previously marked a line in each listing with `=>`.  In a bare \
> `list` this is the next line up for execution.  Previously when requesting a \
> specific location (e.g. `list 5`) the requested line would be marked.  With the \
> patch applied, `list` will only ever mark the next line up for execution.  This is \
>                 consistent with the behavior of GDB and PDB (at least).
> - `EOF` is printed when the repeat setting is on and a bare `list` command follows \
> a listing containing the last source line.  This feature is from PDB; it's a \
> somewhat softer message than the one for an explicit `list` request that's out of \
>                 range.
> - I don't speak Chinese or Japanese, so I've omitted localizations for the new \
> messages in those locales.  However, I updated the help text in both to include the \
> new commands, with the descriptions left empty for now.

This pull request has now been integrated.

Changeset: fe6a2020
Author:    Jakob Cornell <jakob+gpg@jcornell.net>
Committer: Chris Plummer <cjplummer@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/fe6a2020873fe1eb8d4236dc1db3008f485f3195
                
Stats:     560 lines in 10 files changed: 517 ins; 6 del; 37 mod

8271356: Modify jdb to treat an empty command as a repeat of the previous command

Reviewed-by: cjplummer, iklam

-------------

PR: https://git.openjdk.java.net/jdk/pull/5290


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic