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