RFR: 7439: Introduced IConstantPoolExtension for Constant Pools [v5]

Jean-Philippe Bempel jpbempel at openjdk.java.net
Sat Nov 13 21:01:39 UTC 2021


On Sat, 13 Nov 2021 11:07:26 GMT, Michael Südkamp <duke at openjdk.java.net> wrote:

>> Jean-Philippe Bempel has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   formatting
>
> Back to the tests - this time with the JPs clone and branch "pool-extension". I realized that in my Eclipse setup the ConstantPoolExtensionTest class doesn't get compiled unless I do a
> - cd core
> - mvn package
> on console. That's why I obiously got the ClassNotFoundException previously. I realized that when I finally found the "Show command line" option with the class path.
> Now I can run the test but have to do "mvn package" always if I want to make changes, which is obviously very clumsy. 
> Sorry that I report more about my Eclipse setup problems here than regarding the PR...
> 
> Regarding the PR, I have the same test result as last time. I just can't seenany of my own classes in the Parsewr Extension callbacks when I replaced the test JFR file with my own. As I said, I probably didn't got the idea.
> 
> I would suggest that you provide a test JFR file with a custom class like fr.jpbempel.HelloWorld and demonstrate in the test how to get it.

hello @docwarems 
My test is focusing on event Types that referencing constants.

when using the callback:

Object constantRead(long constantIndex, Object constant, String eventTypeId) {
	return constant;
}

you should look at constant argument that should be what you would like to de-obfuscate and return a translated value instead of the original one:


Object constantRead(long constantIndex, Object constant, String eventTypeId) {
	return deobfuscate(constant);
}

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

PR: https://git.openjdk.java.net/jmc/pull/333


More information about the jmc-dev mailing list