clear compiler warnings

[wyli]

Signed-off-by: Wenqi Li wenqil@nvidia.com

Description

clear warnings such as

/MONAI/monai/csrc/filtering/permutohedral/permutohedral_cpu.cpp:469:7: warning: 'delete' applied to a pointer that was allocated with 'new[]'; did you mean 'delete[]'? [-Wmismatched-new-delete]
      delete newValue;
      ^
            []
/MONAI/monai/csrc/filtering/permutohedral/permutohedral_cpu.cpp:417:26: note: allocated with 'new[]' here
    scalar_t* newValue = new scalar_t[vd * hashTable.size()];

Status

Ready

Types of changes

• Non-breaking change (fix or new feature that would not break existing functionality).
• Breaking change (fix or new feature that would cause existing functionality to change).
• New tests added to cover the changes.
• Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
• Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
• In-line docstrings updated.
• Documentation updated, tested make html command in the docs/ folder.

[Nic-Ma]

Looks good to me.
But I am not familiar with our C++ extensions, better to have @charliebudd double confirm it.

Thanks.

[tvercaut]

I think it would be nicer to replace the c arrays by c++ objects (std::vector or std::array when possible). The current memory management seems brittle.

[wyli]

/build