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.

Sign up for GitHub

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

Jump to bottom

Update ExprTk #62

Merged
merged 1 commit into from
Feb 24, 2023
Merged

Update ExprTk #62

merged 1 commit into from
Feb 24, 2023

Conversation

ArashPartow
Copy link
Contributor

@ArashPartow ArashPartow commented Feb 20, 2023 •
edited
Loading

This update adds:

  1. Runtime checks to expressions when being evaluated
  2. Stackoverflow protections during both compilation and evaluation
  3. Various evaluation performance improvements
  4. Code clean-up in prep for rebasing to C++17

@Nicogene
Copy link
Member

Hi @ArashPartow!

Thanks for submitting the PR, could you please fill up the body of the PR explaining the rationale of your changes?
Thanks!

@ArashPartow
Copy link
Contributor Author

@Nicogene it seems the release/debug workflows are stuck, anything I can do at my end?

@traversaro
Copy link
Member

@Nicogene it seems the release/debug workflows are stuck, anything I can do at my end?

Apparently it needed an approval to run, now it should be work fine.

@Nicogene Nicogene merged commit a14bc35 into robotology:master Feb 24, 2023
@Nicogene
Copy link
Member

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nicogene Nicogene Nicogene approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ArashPartow @Nicogene @traversaro