On Tue, 16 Sep 2025 19:23:06 GMT, Alexey Ivanov <[email protected]> wrote:
> Code review https://git.openjdk.org/jdk/pull/27197 for > [JDK-8367348](https://bugs.openjdk.org/browse/JDK-8367348) made me think how > to avoid adding more parameters to methods, in particular > `createInstructionUIPanel`. The `Builder` object captures all the required > configuration data, it is the `Builder` object that should be used to pass > the configuration. > > This changeset refactors UI creation in `PassFailJFrame`. > > * The remaining constructor that accepts positional parameters now creates a > builder to pass the configuration data. > * The `createInstructionUIPanel` method now accepts `Builder` instead of a > set of parameters from it. > * The `createUI` method with positional parameters has become redundant and > is removed. Code duplication between two versions of `createUI` is now > eliminated. > > There are no functional differences. I verified it by launching a few tests > which use `PassFailJFrame` constructors and builder. This pull request has now been integrated. Changeset: d278043d Author: Alexey Ivanov <[email protected]> URL: https://git.openjdk.org/jdk/commit/d278043ddba0cd9ec3ddf8b490366965f5831a22 Stats: 55 lines in 1 file changed: 8 ins; 33 del; 14 mod 8367772: Refactor createUI in PassFailJFrame Reviewed-by: dnguyen, honkar ------------- PR: https://git.openjdk.org/jdk/pull/27321
