Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

Remove usage of assert(0) from library #67

Closed
frobware opened this issue Feb 21, 2017 · 1 comment
Closed

Remove usage of assert(0) from library #67

frobware opened this issue Feb 21, 2017 · 1 comment

Comments

@frobware
Copy link
Contributor

frobware commented Feb 21, 2017 •
edited by yasserf
Loading

There are numerous places where there is an assert(0) which is not great experience as a consumer of the library. The library should either return status codes or throw exceptions, not bomb out (assuming the asserts are compiled in).
@yasserf
Copy link
Contributor

yasserf commented Mar 23, 2017

They are not compiled in, so closing issue.

@yasserf yasserf closed this as completed Mar 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@frobware @yasserf