Skip to content

added three constructors to code_function_callt #2841

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

Merged
merged 1 commit into from
Aug 30, 2018

Conversation

kroening
Copy link
Member

No description provided.

@@ -872,6 +872,36 @@ class code_function_callt:public codet
op2().id(ID_arguments);
}

explicit code_function_callt(const exprt &_function) : codet(ID_function_call)
Copy link
Member

Choose a reason for hiding this comment

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

Mark the no-argument constructor as deprecated.

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@kroening kroening force-pushed the code_function_callt_constructors branch from d989cce to 4724a2b Compare August 27, 2018 13:53
@thk123
Copy link
Contributor

thk123 commented Aug 29, 2018

TG bump (CI running): diffblue/test-gen#2204

@kroening kroening force-pushed the code_function_callt_constructors branch from 4724a2b to 0c4ee80 Compare August 29, 2018 19:49
@tautschnig
Copy link
Collaborator

To fix up CI this requires a rebase once #2863 has been merged.

@kroening kroening force-pushed the code_function_callt_constructors branch from 0c4ee80 to 7583d41 Compare August 30, 2018 13:45
@kroening kroening force-pushed the code_function_callt_constructors branch from 7583d41 to 7c6ea9f Compare August 30, 2018 15:42
@kroening kroening merged commit cdd94b9 into develop Aug 30, 2018
@kroening kroening deleted the code_function_callt_constructors branch August 30, 2018 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants