Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Specify "fill value" directly with variable declarations and during construction of temporaries #1194

Merged
merged 3 commits into from
Jul 10, 2024
Merged

Specify "fill value" directly with variable declarations and during construction of temporaries #1194

merged 3 commits into from
Jul 10, 2024

Commits on Jul 10, 2024

  1. STYLE: Specify "fill value" directly with variable declarations 

    No need to call `variable.Fill(fillValue)` afterwards. By Notepad++, Replace in Files:

    Find what: ([ ]+ )(\w+)([ ]+)(\w+)\(([^ ]+)\);\r\n\1\4\.Fill\(
    Replace with: $1$2$3$4\($5,
    [v] Match case
    (*) Regular expression
    @N-Dekker
    N-Dekker committed Jul 10, 2024
    Configuration menu
    Copy the full SHA
    f6d6f6f View commit details
    Browse the repository at this point in the history

  2. STYLE: Remove redundant Fill(0) MultiResolutionImageRegistrationMethod2 

    No need to "reset" its own Parameters inside its default-constructor
    @N-Dekker
    N-Dekker committed Jul 10, 2024
    Configuration menu
    Copy the full SHA
    f773948 View commit details
    Browse the repository at this point in the history

  3. STYLE: Specify fill value (0) when constructing temporary DerivativeType 

    No need to call `Fill` afterwards, in the ComputeDerivativeLowMemory member functions of ParzenWindowMutualInformationImageToImageMetric.
    @N-Dekker
    N-Dekker committed Jul 10, 2024
    Configuration menu
    Copy the full SHA
    f5f3f9f View commit details
    Browse the repository at this point in the history