projects / folly.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 0671976) | patch
Remove some unnecessary uses of LOG(FATAL) in folly
authorJordan DeLong <jdelong@fb.com>
Fri, 11 Jan 2013 20:52:50 +0000 (12:52 -0800)
committerJordan DeLong <jdelong@fb.com>
Sat, 19 Jan 2013 00:37:56 +0000 (16:37 -0800)
commitaea0f7108838a5a9ff9d30f82638ed9e4a077601
tree921eeaa1cd4d30eccea694e4a42d51d869213845tree | snapshot
parent06719763136dac97720baefbca7b4045afc8386acommit | diff
Remove some unnecessary uses of LOG(FATAL) in folly

Summary:
ScopeGuard::execute will already call std::terminate if an
exception is thrown, as it's noexcept, and the case in
ConcurrentSkipList should be a check.

Test Plan:
Ran folly tests.  Will look at phabricator to see if
anything breaks downstream from removing glog/logging.h from
scopeguard.

Reviewed By: chip@fb.com

FB internal diff: D677038
folly/ConcurrentSkipList.h diff | blob | history
folly/ScopeGuard.h diff | blob | history
folly/test/ScopeGuardTest.cpp diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.