Incorrect use of NDEBUG

allendaicool / thrust

Automatically exported from code.google.com/p/thrust

Apache License 2.0

0 stars 0 forks source link

Incorrect use of NDEBUG #248

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

NDEBUG is used by the standard C library "assert.h" for disabling
assert statements.  See "man assert".

When someone define the macro NDEBUG, they want the code to run fast.
In this case, Thrust shouldn't turn on THURST_DEBUG.

from "thrust/detail/debug.h"

> #ifndef THRUST_DEBUG
> #  if (NDEBUG || DEBUG || _DEBUG)
> #    define THRUST_DEBUG 1
> #  endif // (NDEBUG || DEBUG)
> #endif // THRUST_DEBUG

Original issue reported on code.google.com by heypa...@gmail.com on 8 Oct 2010 at 3:47

GoogleCodeExporter commented 9 years ago

Original comment by jaredhoberock on 8 Oct 2010 at 3:51

Changed state: Accepted
Added labels: Milestone-Release1.4

GoogleCodeExporter commented 9 years ago

This issue was closed by revision 7072bb902a.

Original comment by jaredhoberock on 8 Oct 2010 at 3:56

Changed state: Fixed