Integrated: 8334164: The fix for JDK-8322811 should use _filename.is_set() rather than strcmp()

Sonia Zaldana Calles szaldana at openjdk.org
Thu Jun 20 08:31:18 UTC 2024


On Thu, 13 Jun 2024 16:55:15 GMT, Sonia Zaldana Calles <szaldana at openjdk.org> wrote:

> Hi all, 
> 
> This PR addresses [8334164](https://bugs.openjdk.org/browse/JDK-8334164). 
> 
> Testing: 
> - [x] Verified specifying the literal `vm_memory_map_<pid>.txt` as the map name does not replace `<pid>`. 
> - [x] Verified `<pid>` is still appropriately replaced if a file path is not specified. 
> 
> Thanks, 
> Sonia

This pull request has now been integrated.

Changeset: cabd1046
Author:    Sonia Zaldana Calles <szaldana at openjdk.org>
Committer: Severin Gehwolf <sgehwolf at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/cabd1046d08865f122663d18708d40e5c885c1c3
Stats:     5 lines in 1 file changed: 2 ins; 2 del; 1 mod

8334164: The fix for JDK-8322811 should use _filename.is_set() rather than strcmp()

Reviewed-by: dholmes, cjplummer

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

PR: https://git.openjdk.org/jdk/pull/19706


More information about the serviceability-dev mailing list