Skip to content

Conversation

@kroening
Copy link
Collaborator

@kroening kroening commented Oct 2, 2022

This fixes the removal of function pointers following the API change in 86c8181.

  • Each commit message has a non-empty body, explaining why the change was made.
  • n/a Methods or procedures I have added are documented, following the guidelines provided in CODING_STANDARD.md.
    -n/a The feature or user visible behaviour I have added or modified has been documented in the User Guide in doc/cprover-manual/
  • Regression or unit tests are included, or existing tests cover the modified code (in this case I have detailed which ones those are in the commit message).
  • n/a My commit message includes data points confirming performance improvements (if claimed).
  • My PR is restricted to a single feature or bugfix.
  • n/a White-space or formatting changes outside the feature-related changed lines are in commits of their own.

This fixes the removal of function pointers following the API change in
86c8181.
@kroening kroening added the bugfix label Oct 2, 2022
@codecov
Copy link

codecov bot commented Oct 2, 2022

Codecov Report

Base: 77.82% // Head: 77.82% // No change to project coverage 👍

Coverage data is based on head (2a08d59) compared to base (5e2ff66).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #7183   +/-   ##
========================================
  Coverage    77.82%   77.82%           
========================================
  Files         1578     1578           
  Lines       182242   182242           
========================================
  Hits        141824   141824           
  Misses       40418    40418           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@kroening kroening marked this pull request as ready for review October 2, 2022 19:37
@kroening kroening merged commit 1383819 into develop Oct 3, 2022
@kroening kroening deleted the fix-function-pointer-removal branch October 3, 2022 06:49
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.

3 participants