-
Notifications
You must be signed in to change notification settings - Fork 15
Open
Description
Hi,
Just a small suggestion. It is more reasonable to load config only if the original config is null, and not load always and then comparing both (#See Optional.orElse vs orElseGet) :)
headless-chrome/src/main/java/software/reinvent/headless/chrome/HeadlessDriver.java
Line 36 in 3758865
| Config configToUse = Optional.ofNullable(config).orElse(ConfigLoader.load()); |
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels