Skip to content

Fix rapidjson assert#429

Merged
jcoupey merged 3 commits intomasterfrom
fix/rapidjson-assert
Jan 13, 2021
Merged

Fix rapidjson assert#429
jcoupey merged 3 commits intomasterfrom
fix/rapidjson-assert

Conversation

@jcoupey
Copy link
Copy Markdown
Collaborator

@jcoupey jcoupey commented Jan 7, 2021

Issue

Fixes #418

Tasks

  • Check first vehicle validity before any call to get_amount_size
  • Avoid looping on vehicles to get amount size
  • Update CHANGELOG.md
  • review

@jcoupey jcoupey added this to the v1.9.0 milestone Jan 7, 2021
@jcoupey jcoupey merged commit 5cfa906 into master Jan 13, 2021
@jcoupey jcoupey deleted the fix/rapidjson-assert branch January 13, 2021 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rapidjson assert on invalid vehicle syntax

1 participant