projects / folly.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 3f4a8ae) | patch
correctly handle APIs that accept Poly<T&> as an argument
authorEric Niebler <eniebler@fb.com>
Fri, 12 Jan 2018 21:55:50 +0000 (13:55 -0800)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Fri, 12 Jan 2018 22:05:10 +0000 (14:05 -0800)
commit4a5ba093e7ab8eac7354180d749a322de60f7fa9
treede2a900b8c16f4df987523f2ec36c805faaa0620tree | snapshot
parent3f4a8ae005738662fbbca4a85c959df2bf0d059bcommit | diff
correctly handle APIs that accept Poly<T&> as an argument

Summary: Multi-dispatch in Poly was handled by treating arguments of type Poly<I&> as special, unwrapping them. That's a problem if the underlying API actually wants the Poly<I&> unmolested.

Reviewed By: yfeldblum

Differential Revision: D6713975

fbshipit-source-id: 18a90fa701fab14c3d3d46c247efe09ea5903b11
folly/detail/PolyDetail.h diff | blob | history
folly/test/PolyTest.cpp diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.