> Add "findsym" to clhsdb. See the CR and CSR for details. The > [CSR](https://bugs.openjdk.java.net/browse/JDK-8261101) still needs a > reviewer. > > There is a fix in LinuxDebuggerLocal_lookupByName0() to allow passing in NULL > for the dso name. This is allowed, and in fact even not null it gets ignored. > It is needed by the new findsym support in order for the following to work, > which passes in null for the dso name: > > ` Address addr = VM.getVM().getDebugger().lookup(null, symbol);` > > There is one other somewhat unrelated fix in the test: > > - String value = parts[1]; > + String value = parts[1].split(linesep)[0]; > > This is suppose to capture just the value at the specified address, but it > also captures the newline and some text after. The result if that when > `findpc <value>` is executed, it also executes another command or two of > garbage commands after that, which produce errors. They were not impacting > the test, but were noticeable in the log. I first noticed it when similar > code in the new part of the test had the same issue.
Chris Plummer has updated the pull request incrementally with one additional commit since the last revision: Fix issue with findsym on windows build that doesn't have symbolic information. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/2567/files - new: https://git.openjdk.java.net/jdk/pull/2567/files/8c6d9727..ba3464c4 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2567&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2567&range=00-01 Stats: 29 lines in 1 file changed: 15 ins; 1 del; 13 mod Patch: https://git.openjdk.java.net/jdk/pull/2567.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2567/head:pull/2567 PR: https://git.openjdk.java.net/jdk/pull/2567
