Skip to content

Refactor conversions from primitive types to String #27

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 3 commits into from
Sep 9, 2019

Conversation

danpoe
Copy link
Contributor

@danpoe danpoe commented Aug 21, 2019

No description provided.

@romainbrenguier
Copy link
Contributor

There should be first a pull request on CBMC to add the CProverString.toString function to the preprocessing (can be merged without this one if it includes some test). Then this one can be merged (there should also be a similar one for the models library). Then you can make one in CBMC to remove the preprocessing of the non-CProverString functions.

@danpoe danpoe force-pushed the danpoe/refactor/primitive-types-to-string branch 3 times, most recently from 7b8201d to 10e906c Compare August 28, 2019 13:06
danpoe added a commit to danpoe/cbmc that referenced this pull request Aug 28, 2019
danpoe added a commit to danpoe/cbmc that referenced this pull request Aug 29, 2019
danpoe added a commit to danpoe/cbmc that referenced this pull request Aug 30, 2019
@danpoe danpoe force-pushed the danpoe/refactor/primitive-types-to-string branch 2 times, most recently from 1fc7752 to d47727c Compare September 9, 2019 10:55
danpoe added a commit to danpoe/cbmc that referenced this pull request Sep 9, 2019
danpoe added a commit to danpoe/cbmc that referenced this pull request Sep 9, 2019
@danpoe danpoe removed the don't merge label Sep 9, 2019
@danpoe danpoe merged commit b545958 into master Sep 9, 2019
@rjmunro rjmunro deleted the danpoe/refactor/primitive-types-to-string branch September 11, 2019 09:24
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.

3 participants