On Wed, 9 Aug 2023 21:02:37 GMT, Harshitha Onkar <[email protected]> wrote:
>> lawrence.andrews has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Changed enableScreenCapture method name to screenCapture shorter method >> name >> - Update full name > > test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 581: > >> 579: private int rows; >> 580: private int columns; >> 581: private boolean enableScreenCapture = false; > > @lawrence-andrew you missed updating var `enableScreenCapture` to > `screenCapture` here. You can either update only the method name to > screenCapture() and retain same var name (`enableScreenCapture`) or update > two places where they are being used. Its better to keep method name and var name separately. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14094#discussion_r1289349317
