Apply suggestions from code review

these were reverted changes that got missed after the renaming rebase Co-authored-by: Matthew Marine <matthew.marine@bluequartz.net>
BlueQuartzSoftware · May 1, 2024 · 0ecdb4c · 0ecdb4c
1 parent 6384ad7
commit 0ecdb4c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...ins/ITKImageProcessing/src/ITKImageProcessing/Filters/Algorithms/ITKImportFijiMontage.cpp b/...ins/ITKImageProcessing/src/ITKImageProcessing/Filters/Algorithms/ITKImportFijiMontage.cpp
@@ -269,7 +269,7 @@ class IOHandler
         // rename grayscale array to reflect original
         {
           auto& gray = m_DataStructure.getDataRefAs<IDataArray>(imageDataPath.replaceName("gray" + imageDataPath.getTargetName()));
-          if(gray.canRename(imageDataPath.getTargetName()) != 1)
+          if(gray.canRename(imageDataPath.getTargetName()) == false)
           {
             return MakeErrorResult(-18543, fmt::format("Unable to rename the grayscale array to {}", imageDataPath.getTargetName()));
           }

diff --git a/src/simplnx/DataStructure/DataObject.hpp b/src/simplnx/DataStructure/DataObject.hpp
@@ -205,7 +205,7 @@ class SIMPLNX_EXPORT DataObject
    * @brief Checks and returns if the DataObject can be renamed to the provided
    * value.
    * @param name
-   * @return int: 0 = false, 1 = true, 2 = duplicate object found
+   * @return bool
    */
   bool canRename(const std::string& name) const;