Cut throwOnFail
authorYedidya Feldblum <yfeldblum@fb.com>
Wed, 26 Jul 2017 07:04:24 +0000 (00:04 -0700)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Wed, 26 Jul 2017 07:13:31 +0000 (00:13 -0700)
commitc815830fdb18d1b27bbd9bc883f16935759c0a64
tree194c87875d654e08f4a1ead099d158a105166038
parent5a40209ae92c09d6c13875e5ce162cab5456c540
Cut throwOnFail

Summary:
[Folly] Cut `throwOnFail`.

It is not necessary, and `CHECK_THROW`, its reason for existing, can be implemented without it.

It can also easily be a pessimization because there is no way to delay computation of the arguments to the exception ctor until after the check. So if there is, say, a computation using `sformat` to compute a string argument to the exception ctor, that will always be performed rather than being performed only should the check fail.

Reviewed By: Orvid

Differential Revision: D5478804

fbshipit-source-id: 71a125c126eae76c6e95ef1bd23ee883b1db39a5
folly/Exception.h