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

Support for set<> type #123

Closed
wants to merge 4 commits into from
Closed

Support for set<> type #123

wants to merge 4 commits into from

Conversation

mihaibudiu
Copy link
Contributor

No description provided.

@@ -177,9 +177,7 @@ FILE* CompilerOptions::preprocess() {
void CompilerOptions::closeInput(FILE* inputStream) const {
if (close_input) {
int exitCode = pclose(inputStream);
if (WIFEXITED(exitCode) && WEXITSTATUS(exitCode) == 4)
::error("input file %s does not exist", file);
else if (exitCode != 0)
Copy link
Contributor

Choose a reason for hiding this comment

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

why remove this change?

@mihaibudiu
Copy link
Contributor Author

I think that this got somehow lost when I did the git reset.

@ChrisDodd
Copy link
Contributor

I merged this (except for the options change), we'll see if github picks up on that.

@mihaibudiu
Copy link
Contributor Author

You have merged but not closed, so my new commits went on top of the existing pull request again...
This change puts error in its own namespace, fixing issue #98

@ChrisDodd
Copy link
Contributor

merged as d11a786

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.

2 participants