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

Remove objexx gio 4 #7727

Merged
merged 9 commits into from
Feb 3, 2020
Merged

Remove objexx gio 4 #7727

merged 9 commits into from
Feb 3, 2020

Conversation

lefticus
Copy link
Contributor

@lefticus lefticus commented Jan 29, 2020

Pull request overview

Phase four of objexx gio removal.

Focuses only on InternalHeatGains.cc due to the number of changes.

Note that this reduced the file by ~500LOC

The two EIO diffs are clear bug fixes compared to the original file (adding a missing , specifically)

@lefticus
Copy link
Contributor Author

@Myoldmopar I don't know why the Windows build failed to post to CI, but I do have a potential E+ error on Windows I need to investigate. FYI

@lefticus
Copy link
Contributor Author

@Myoldmopar the error is a floating point underflow that happens in debug builds with a few models. Doesn't seem to be related to any changes I made.

@mitchute
Copy link
Collaborator

mitchute commented Feb 3, 2020

This looks great. EIO diffs are legitimate bug fixes. It's up to date with develop. Merging.

@mitchute mitchute merged commit 8478cc4 into develop Feb 3, 2020
@mitchute mitchute deleted the remove_objexx_gio__4 branch February 3, 2020 15:59
@Myoldmopar Myoldmopar added the Refactoring Includes code changes that don't change the functionality of the program, just perform refactoring label Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactoring Includes code changes that don't change the functionality of the program, just perform refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants