opm-common
opm-common copied to clipboard
Varnish WELSPECS Report Implementation
In particular,
- Split long lines.
- Switch to initialising objects, especially references, using assignment syntax (
=) instead of construction ({}). - Move function opening braces to next line.
- Remove blanks before semicolons.
- Apply
constwhere reasonable. - Move static objects into function-local scopes where reasonable.
- Reverse conditionals to enable "early continue".
- Rename
workers::write_WELSPECS()toworkers::wellSpecification().
This is in preparation to replacing hard-coded date stamps with dynamic information.
I'm creating this PR in draft mode because it depends on, and contains, the earlier PR #4526. I will keep the PR in a draft state until such time as it is ready for review and merging.
The upstream PR was merged into the master branch. I'm marking this work as "ready for review".
jenkins build this please
jenkins build this please
PR approved and build check is green. I'll merge into master.