llnl / llnl/AMS

Revamp AMS debug THROW

Open
#206 0 comments 0 reactions 1 assignee Claimed by @lpottier View on GitHub
bug
Dominant language
C++
Stars
10
Forks
8
Avg merge
17d 21h
Merged PRs (30d)
1

Description

- `THROW` in `wf/debug.h` does not throw but instead `abort()` which can lead to issues down the line.
- `AMS_WARNING` compiles to nothing when `__AMS_DEBUG__` which might not the desired behavior (you want warning also in release mode)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.