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.

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

crash when trying to load int64 or uint64 volume #32

Closed
slicerbot opened this issue Mar 12, 2020 · 0 comments
Closed

crash when trying to load int64 or uint64 volume #32

slicerbot opened this issue Mar 12, 2020 · 0 comments

Comments

@slicerbot
Copy link
Collaborator

This issue was created automatically from an original Mantis Issue. Further discussion may take place here.
lassoan added a commit to lassoan/Slicer that referenced this issue Aug 13, 2020
@lassoan
ENH: Update to latest SurfaceToolbox
83eeeb8 
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
@lassoan
ENH: Update to latest SurfaceToolbox
9e2d2ac 
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
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@slicerbot