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
crash when trying to load int64 or uint64 volume #32
Comments
lassoan
added a commit
to lassoan/Slicer
that referenced
this issue
Aug 13, 2020
Contains fixes of warnings and tests and update of SurfaceToolbox module to follow latest scripted module template. Details: Revision: 28b2567f8c155d4665ae5c684c5d8ad9e7961934 Author: Andras Lasso <lasso@queensu.ca> Date: 2020-08-13 12:59:28 AM Message: BUG: Update SurfaceToolbox to latest scripted module template and fix test SurfaceToolbox.py now follows latest scripted module template, it is simplified, and its size is reduced to about half. Revision: c6dab36d20af0a43b738dff682b6b5c07fac2553 Author: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com> Date: 2020-07-30 6:40:23 PM Message: Merge pull request Slicer#32 from jcfr/fix-unused-var-warnings COMP: Fix -Wunused-variable warnings in qSlicerDynamicModelerModuleWidget Revision: ff86d59d22f4231b0229b6a3fc87d072c6215977 Author: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com> Date: 2020-07-30 5:51:03 PM Message: COMP: Fix -Wunused-variable warnings in qSlicerDynamicModelerModuleWidget This commit fixes the following warnings: /path/to/Slicer-Release/SurfaceToolbox/DynamicModeler/qSlicerDynamicModelerModuleWidget.cxx: In member function ‘void qSlicerDynamicModelerModuleWidget::updateMRMLFromWidget()’: /path/to/Slicer-Release/SurfaceToolbox/DynamicModeler/qSlicerDynamicModelerModuleWidget.cxx:681:9: warning: unused variable ‘inputIndex’ [-Wunused-variable] int inputIndex = inputNodeSelector->property("InputIndex").toInt(); ^ /path/to/Slicer-Release/SurfaceToolbox/DynamicModeler/qSlicerDynamicModelerModuleWidget.cxx:682:9: warning: unused variable ‘inputSelectorIndex’ [-Wunused-variable] int inputSelectorIndex = inputNodeSelector->property("InputSelectorIndex").toInt(); ^
lassoan
added a commit
that referenced
this issue
Aug 13, 2020
Contains fixes of warnings and tests and update of SurfaceToolbox module to follow latest scripted module template. Details: Revision: 28b2567f8c155d4665ae5c684c5d8ad9e7961934 Author: Andras Lasso <lasso@queensu.ca> Date: 2020-08-13 12:59:28 AM Message: BUG: Update SurfaceToolbox to latest scripted module template and fix test SurfaceToolbox.py now follows latest scripted module template, it is simplified, and its size is reduced to about half. Revision: c6dab36d20af0a43b738dff682b6b5c07fac2553 Author: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com> Date: 2020-07-30 6:40:23 PM Message: Merge pull request #32 from jcfr/fix-unused-var-warnings COMP: Fix -Wunused-variable warnings in qSlicerDynamicModelerModuleWidget Revision: ff86d59d22f4231b0229b6a3fc87d072c6215977 Author: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com> Date: 2020-07-30 5:51:03 PM Message: COMP: Fix -Wunused-variable warnings in qSlicerDynamicModelerModuleWidget This commit fixes the following warnings: /path/to/Slicer-Release/SurfaceToolbox/DynamicModeler/qSlicerDynamicModelerModuleWidget.cxx: In member function ‘void qSlicerDynamicModelerModuleWidget::updateMRMLFromWidget()’: /path/to/Slicer-Release/SurfaceToolbox/DynamicModeler/qSlicerDynamicModelerModuleWidget.cxx:681:9: warning: unused variable ‘inputIndex’ [-Wunused-variable] int inputIndex = inputNodeSelector->property("InputIndex").toInt(); ^ /path/to/Slicer-Release/SurfaceToolbox/DynamicModeler/qSlicerDynamicModelerModuleWidget.cxx:682:9: warning: unused variable ‘inputSelectorIndex’ [-Wunused-variable] int inputSelectorIndex = inputNodeSelector->property("InputSelectorIndex").toInt(); ^
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This issue was created automatically from an original Mantis Issue. Further discussion may take place here.
The text was updated successfully, but these errors were encountered: