Skip to content

Restore recognition of dynamic objects #4537

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

Conversation

smowton
Copy link
Contributor

@smowton smowton commented Apr 15, 2019

These were accidentally disabled when distinguishing ID_is_dynamic_object (a predicate that tests
whether an object is dynamic) from ID_dynamic_object (a reference to the object itself, similar to
symbol_exprt). I also take the opportunity to restore pretty-printing of dynamic object expressions
(while also keeping pretty-printing of the predicate).

This partially reverts #4479

@owen-mc-diffblue
Copy link
Contributor

Clang-format and the linter are failing

These were accidentally disabled when distinguishing ID_is_dynamic_object (a predicate that tests
whether an object is dynamic) from ID_dynamic_object (a reference to the object itself, similar to
symbol_exprt). I also take the opportunity to restore pretty-printing of dynamic object expressions
(while also keeping pretty-printing of the predicate).
@smowton smowton force-pushed the smowton/fix/restore-dynamic-object branch from 86cf481 to 9014506 Compare April 15, 2019 17:03
Copy link
Contributor

@allredj allredj left a comment

Choose a reason for hiding this comment

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

✔️
Passed Diffblue compatibility checks (cbmc commit: 9014506).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/108401786

@tautschnig
Copy link
Collaborator

tautschnig commented Apr 15, 2019

How did you notice? Would it be possible to have a test? (Edit: I should have read the PR first, which does include a unit test.)

@tautschnig tautschnig merged commit 6724b09 into diffblue:develop Apr 15, 2019
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.

4 participants