Skip to content

[Python-builds] Cleanup python test#644

Closed
maekclena wants to merge 8 commits intoInsightSoftwareConsortium:python-buildsfrom
maekclena:cleanup_python_test
Closed

[Python-builds] Cleanup python test#644
maekclena wants to merge 8 commits intoInsightSoftwareConsortium:python-buildsfrom
maekclena:cleanup_python_test

Conversation

@maekclena
Copy link
Copy Markdown
Contributor

Python builds for #643.

Sorry for the other commits, I rebased from master.

N-Dekker and others added 7 commits March 27, 2019 17:29
Simplified `ImageFileReader::GenerateData()` implementation by declaring
its local `loadBuffer` as `const std::unique_ptr<char[]>`, instead of as
a raw `char` pointer.

Removed try-catch-throw and `delete[]` statements that are no longer necessary
now, due to the C++ "RAII" idiom implemented by `std::unique_ptr`.

Replaced a `std::copy` by `std::copy_n`, to further simplify the code.
Add a script to update the remote module commit hashes.

Update the `Release.md` guide to add the remote module update step prior
to a release.
Add a script to update the ITK tag as required in
- `azure-pipelines.yaml` Azure pipelines configuration file.
- `setup.py` Python setup files.

usually in the `Remote` modules.
STYLE: ImageFileReader uses std::unique_ptr<[]>, try-catch-throw removed
ENH: Add a script to update the remote module commit hashes
ENH: Add a script to update the ITK tag as required in files
Copy link
Copy Markdown
Member

@thewtex thewtex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @maekclena ! 👍 🥇

@fbudin69500
Copy link
Copy Markdown
Contributor

Merged #643 into master.

@maekclena maekclena deleted the cleanup_python_test branch April 15, 2019 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants