-
Notifications
You must be signed in to change notification settings - Fork 308
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce and use ScanSummary.EMPTY
#5937
Conversation
Codecov ReportBase: 57.95% // Head: 57.95% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #5937 +/- ##
=========================================
Coverage 57.95% 57.95%
Complexity 2057 2057
=========================================
Files 322 322
Lines 18666 18666
Branches 3759 3759
=========================================
Hits 10817 10817
Misses 6802 6802
Partials 1047 1047
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
ab423c4
to
17aae9d
Compare
17aae9d
to
2994ec1
Compare
The constant can be used with the `copy` function to conveniently create an instance with only some values set. Signed-off-by: Frank Viernau <frank_viernau@epam.com>
Slightly simplify an upcoming change. Signed-off-by: Frank Viernau <frank_viernau@epam.com>
Improve readability of the caller code. Signed-off-by: Frank Viernau <frank_viernau@epam.com>
Signed-off-by: Frank Viernau <frank_viernau@epam.com>
Use named arguments, inline a variable to end up with an expression function. Signed-off-by: Frank Viernau <frank_viernau@epam.com>
The test does cover the values used for these times. So, use `Instant.EPOCH` for making the value deterministic, for aligning with other "empty" default values. While at it also inline the variable to simplify an upcoming change. Signed-off-by: Frank Viernau <frank_viernau@epam.com>
Eliminate repeating default values by removing constructor calls in favor of using `ScanSummary.EMPTY.copy()`. Signed-off-by: Frank Viernau <frank_viernau@epam.com>
2994ec1
to
9b63690
Compare
This eliminates repetition of default values in scan summary constructions.
It prepares for further refactoring steps.
Part of #5681.