RFR: 7903519 : jtreg/jtharness is missing features for basic crash testing [v4]

andrlos duke at openjdk.org
Wed Dec 13 21:17:11 UTC 2023


On Wed, 13 Dec 2023 20:48:44 GMT, andrlos <duke at openjdk.org> wrote:

>> src/com/sun/javatest/exec/ResultModifierInterface.java line 37:
>> 
>>> 35:  * crashed in the process without being intended to.
>>> 36:  */
>>> 37: public interface ResultModifierInterface {
>> 
>> Suggestion:
>> 
>> public interface ResultModifier {
>> 
>> 
>> Should it be just `ResultModifier` rather than `ResultModifierInterface`?
>
> I am under impression that naming interface with suffix "Interface" is kinda a good practice..

but if you insist on renaming it I have no problem with it :-)

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

PR Review Comment: https://git.openjdk.org/jtharness/pull/57#discussion_r1425891618


More information about the jtharness-dev mailing list