Do not mess with NDEBUG in fbstring
Do not mess with NDEBUG in fbstring

Summary: Temporarily overriding `NDEBUG` can have unexpected side-effects depending on the implementation of `<assert.h>`. Better use an explicit macro for `assert()` that we can explicitly disable.

Reviewed By: Gownta

Differential Revision: D3850717

fbshipit-source-id: b1e7fce89ab4120b0224e9c6bf668d983b3d2d31

floxard/folly,charsyam/folly,facebook/folly,rklabs/folly,Orvid/folly,Orvid/folly,facebook/folly,floxard/folly,charsyam/folly,facebook/folly,facebook/folly,floxard/folly,floxard/folly,Orvid/folly,charsyam/folly,rklabs/folly,Orvid/folly,rklabs/folly,rklabs/folly,Orvid/folly,floxard/folly,facebook/folly,rklabs/folly,charsyam/folly