8000 Need to evaluate all asserts in code base · Issue #29 · stratis-storage/project · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Need to evaluate all asserts in code base #29

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
Closed
1 task
mulkieran opened this issue Feb 23, 2018 · 4 comments
Closed
1 task

Need to evaluate all asserts in code base #29

mulkieran opened this issue Feb 23, 2018 · 4 comments
Assignees
@mulkieran @bgurney-rh

Comments

@mulkieran
Copy link
Member
mulkieran commented Feb 23, 2018 •
edited
Loading

Need to decide which should be asserts, and which should be debug asserts, which get compiled out.
@mulkieran
Copy link
Member Author

Need to get rid of ones that should actually be errors, and not asserts at all.

@mulkieran mulkieran self-assigned this Jun 14, 2018
@mulkieran
Copy link
Member Author

It turns out I am working on this, but mostly in devicemapper, see:
stratis-storage/devicemapper-rs#417, merged
stratis-storage/devicemapper-rs#430, in progress.

@mulkieran mulkieran transferred this issue from stratis-storage/stratisd May 17, 2019
This was referenced May 17, 2019
Closed
Merged
@mulkieran
Copy link
Member Author

stratis-storage/devicemapper-rs#484 also falls into this category.

@mulkieran mulkieran assigned bgurney-rh and unassigned waltdisgrace Dec 19, 2019
@mulkieran mulkieran mentioned this issue Jan 15, 2020
Merged
@mulkieran
Copy link
Member Author

We do this all the time these days, as part of our regular operation, so we can drop this as a special issue.

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

@mulkieran mulkieran

@bgurney-rh bgurney-rh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mulkieran @bgurney-rh @waltdisgrace
0