Fail on implicit convert from BOOL on ObjC in folly::dynamic
authorMarc Horowitz <mhorowitz@fb.com>
Tue, 18 Apr 2017 22:21:35 +0000 (15:21 -0700)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Tue, 18 Apr 2017 22:34:33 +0000 (15:34 -0700)
commitc31d639e15ceb2a1fab05f1318f886df7c18bbcb
tree3e8614630746c6d9a7f07fdf873baaa76d68e355
parentf258ec23413c8b75680291ce3bb01fc9fed72fc5
Fail on implicit convert from BOOL on ObjC in folly::dynamic

Summary: On some platforms (iPhone 5C at least), ObjC/Objc++ BOOL is really signed char.  There is code which expects this to be a boolean when converted to dynamic (and then to JSON and into JS), but the old code treated it as a number.  This makes such code (like [mobileConfig getBool:]) fail to compile, so the developer needs to resolve the ambiguity one way or the other.

Reviewed By: yfeldblum

Differential Revision: D4648133

fbshipit-source-id: 76ece7803a1e966dca08bdb857af7990035544a0
folly/Portability.h
folly/dynamic-inl.h