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

Fixes for crashes when running doc tests on Windows #846

Merged
merged 14 commits into from Jun 4, 2015

Conversation

martyngigg
Copy link
Member

The Windows debugger caught several out-of-bounds errors that were causing the documentation tests to segfault occasionally. These fixes should cure that.

Tester

Observe that the documentation tests are passing.

The Y array is one shorter that the X array and the Y value corresponding
to the X value is on the left.
@martyngigg martyngigg added In Progress Documentation Issues and pull requests related to documentation labels Jun 2, 2015
@martyngigg martyngigg added this to the Release 3.5 milestone Jun 2, 2015
It should be possible to end up with a zero dimension array from a
workspace.
The output now includes the integrated dimensions.
They always crash on the CI server and need to be investigated.
@OwenArnold OwenArnold self-assigned this Jun 3, 2015
@OwenArnold
Copy link
Contributor

Code changes look good. Just waiting for system tests to pass.

@martyngigg
Copy link
Member Author

Looks like there was a change to LoadCalFile on master so the result needs updating. I'll do that.

OwenArnold added a commit that referenced this pull request Jun 4, 2015
Fixes for crashes when running doc tests on Windows
@OwenArnold OwenArnold merged commit fe4be80 into master Jun 4, 2015
@OwenArnold OwenArnold deleted the fix_doctest_crashes branch June 4, 2015 08:54
@AndreiSavici AndreiSavici added the Patch Candidate Urgent issues that must be included in a patch following a release label Jul 17, 2015
@peterfpeterson peterfpeterson restored the fix_doctest_crashes branch August 10, 2015 15:49
@peterfpeterson peterfpeterson deleted the fix_doctest_crashes branch August 10, 2015 15:49
@peterfpeterson peterfpeterson removed the Patch Candidate Urgent issues that must be included in a patch following a release label Aug 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Issues and pull requests related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants