projects / folly.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 632fb9b) | patch
Recommend a different exception_handler idiom
authorChad Parry <cparry@fb.com>
Wed, 2 Dec 2015 07:36:55 +0000 (23:36 -0800)
committerfacebook-github-bot-1 <folly-bot@fb.com>
Wed, 2 Dec 2015 08:20:21 +0000 (00:20 -0800)
commit266ccb57e47f51a43dd01e19abcf7ec568b07d17
tree24cec185c18442f245d09bcfa0782bb48e0cfeb5tree | snapshot
parent632fb9b591ebc15229f9d293a704c42d45d96535commit | diff
Recommend a different exception_handler idiom

Summary: This is a //tiny// documentation change. I noticed that there are lots of places in the codebase where people follow this recommended usage of `with_exception` in an if–else chain, (e.g., https://fburl.com/181606454). IMHO, a short-circuiting `||` operator is easier to read and write.

Reviewed By: yfeldblum

Differential Revision: D2687840

fb-gh-sync-id: 6354555aeba81dcfe7adf17e7e573de0b1206b37
folly/ExceptionWrapper.h diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.