- 23 Nov, 2021 2 commits
-
-
Alex Hornby authored
Summary: fix a lint, platform variable was declared twice Reviewed By: Croohand Differential Revision: D32605614 fbshipit-source-id: 8e43de8689bf30410273355444db9c3d94663d43
-
Yedidya Feldblum authored
Summary: The stdlibs export ways to detect which are in use. Cut the unnecessary preprocessor names for detecting/overriding. Differential Revision: D32515501 fbshipit-source-id: 2bc85cda30ebbdfa0163e3fe287aceb8df923303
-
- 22 Nov, 2021 2 commits
-
-
Alex Hornby authored
Summary: The project as moved out of github facebookincubator org to facebook org Reviewed By: StanislavGlebik Differential Revision: D32593002 fbshipit-source-id: 4ea29dc63fa60e9a9a4f169ea65f7fdca6333005
-
Alex Hornby authored
Summary: Most referenced repos have moved from master to main as default branch, so flip the default in getdeps. As well as tidying things up, this means more manifests will be able to use the pinned git commit hash for a dependency that shipit generates (e.g. build/deps/github_hashes/facebook/folly-rev.txt). This is because a "rev = " setting in the manifest is treated as a dynamic override in GitFetcher.__init__ and this diff reduces the number of such overrides. Reviewed By: StanislavGlebik Differential Revision: D32567564 fbshipit-source-id: 3f1b734eb1bb0f5dbafb01ceb935a3d51b8dcc9a
-
- 19 Nov, 2021 2 commits
-
-
Shai Szulanski authored
Reviewed By: capickett Differential Revision: D32375291 fbshipit-source-id: 4e88bd4b0caae41c84ef7c249b3108bed9afa80a
-
Shai Szulanski authored
Summary: Provies single-await co_nothrow wrapper Reviewed By: capickett Differential Revision: D32470528 fbshipit-source-id: 30434b133e282f1015f97025c448bb88a1701e7a
-
- 18 Nov, 2021 4 commits
-
-
Jolene Tan authored
Summary: Other template classes, as well as the docstring, use `Ex` to refer to the exception class. Differential Revision: D32192455 fbshipit-source-id: 664db8beee0850ac3fb8e3cfcd46a28d568cb808
-
Mark Santaniello authored
Summary: Enable auto-reserve for operator=(std::initializer_list). Reviewed By: yfeldblum, shixiao Differential Revision: D32233038 fbshipit-source-id: ae16a5508dcae5957ea44404ac55d489a14937af
-
Alex Hornby authored
Summary: libffi is needed for getdeps python 3.8 update Reviewed By: krallin Differential Revision: D32459310 fbshipit-source-id: 6515eed088fb6de0091c2348b5732d1b635113e0
-
Chad Austin authored
Summary: ftp.pcre.org is dead. We are currently migrating from PCRE1 to PCRE2. In the meantime, download from SourceForge instead like we do with Boost. Reviewed By: akushner, xavierd Differential Revision: D32520283 fbshipit-source-id: 9fda6930d7d8b677786a43419c4c0dc6289305f1
-
- 17 Nov, 2021 4 commits
-
-
Andrew Gallagher authored
Summary: In the vast majority of cases in fbcode, the sanitizer macros are checked with `#ifdef`, so do the same in some places to fix `-Wundef`. Reviewed By: luciang Differential Revision: D32501662 fbshipit-source-id: 123e22d78476c9db0e328ccb01a734a06a163594
-
Andrew Gallagher authored
Summary: Users may set `FOLLY_USE_LIBCPP` or `FOLLY_USE_LIBSTDCPP` to `0` or `1`, so make sure everything checks it via `#if`. Reviewed By: luciang Differential Revision: D32496561 fbshipit-source-id: 5eee660b540af0bf1eaee4949576acf11c9be8d0
-
Yedidya Feldblum authored
Summary: Cast the result of shifts to the integer type parameter, rather than the shift width. Reviewed By: aary, luciang Differential Revision: D32160950 fbshipit-source-id: 0100a1a92c48bfa1cd957cf42ed84d01302ea4f7
-
Andrii Grynenko authored
Summary: Previously update() could be run both directly from within the update notification callback (that can happen in any context while holding some application locks), but also while observer is initially created from the observable. This change moves the update logic to be called on the observer internal thread, making the implementation even more resilient to various locks that may be held by the application. Reviewed By: yfeldblum Differential Revision: D32338088 fbshipit-source-id: 557f47c98469a951ce4b5513ce96e2b3a5d8ba6b
-
- 16 Nov, 2021 3 commits
-
-
Andrew Gallagher authored
Reviewed By: igorsugak Differential Revision: D32415926 fbshipit-source-id: b62ac87215800eed8e6cf8acf92f0518b9a56da5
-
Andrew Gallagher authored
Reviewed By: ot Differential Revision: D31763384 fbshipit-source-id: ef666193f83a16d4eedf65a499361d967809163f
-
Abishek Sethuraman authored
Differential Revision: D32405920 (https://github.com/facebook/folly/commit/6b58b9e81875d92ba53add248d113593b24d2c8b) Original commit changeset: b0b5c55149ae fbshipit-source-id: f5a6d0dd6bb4434cebe02b5cde357d94e28253f0
-
- 15 Nov, 2021 3 commits
-
-
Shai Szulanski authored
Summary: Provies single-await co_nothrow wrapper Reviewed By: capickett Differential Revision: D32405920 fbshipit-source-id: b0b5c55149aebe4246422fabeffb7ed433b4e20d
-
Andrii Grynenko authored
Summary: Have constructor do init() instead. I can't think of any valid reason to have this API other than confusing everyone who decides to use it. Reviewed By: praihan Differential Revision: D32302228 fbshipit-source-id: 136e0d2919291acd60be953337309cd006bd674b
-
Pranjal Raihan authored
Summary: Now we'll ignore all the known sanitizers: ASAN, TSAN, MSAN, UBSAN. Reviewed By: yfeldblum, jdonald Differential Revision: D32364717 fbshipit-source-id: f71a294dd9ebaa8521f0487ebc65c6f7c1e7bf04
-
- 13 Nov, 2021 1 commit
-
-
Michel Salim authored
Summary: Inlining `getStackTraceInPlace` fails on Fedora with ``` /builddir/build/BUILD/folly-2020.12.21.00/folly/experimental/symbolizer/StackTrace.cpp: In function 'folly::symbolizer::(anonymous namespace)::getStackTraceInPlace(ucontext_t&, unw_cursor&, unsigned long*, unsigned long)': /builddir/build/BUILD/folly-2020.12.21.00/folly/experimental/symbolizer/StackTrace.cpp:88:9: error: function 'folly::symbolizer::(anonymous namespace)::getStackTraceInPlace(ucontext_t&, unw_cursor&, unsigned long*, unsigned long)' can never be inlined because it uses setjmp 88 | ssize_t getStackTraceInPlace( | ^~~~~~~~~~~~~~~~~~~~ ``` Reviewed By: yfeldblum Differential Revision: D29639503 fbshipit-source-id: 823692d95171d5fb3968761c00b4cc5650db171b
-
- 12 Nov, 2021 4 commits
-
-
Shai Szulanski authored
Differential Revision: D32394449 fbshipit-source-id: 2c2bc54c2d71cc677c35195adc73f579b5d0c101
-
Shai Szulanski authored
Summary: exception_wrapper no longer rethrows during this construction Reviewed By: yfeldblum Differential Revision: D32371738 fbshipit-source-id: 0d5d51cdde4115aaf8774950e53bfac0b37f0b88
-
Shai Szulanski authored
Summary: Breaks rescheduling on error Differential Revision: D32398162 fbshipit-source-id: 582db297e0b700617c2e455e1f6d2fd1fa40e143
-
Christoph Purrer authored
Summary: - For Windows in xplat we use [Clang9/12 + MSVC 2017](https://www.internalfb.com/code/search?q=filepath%3Afbsource%2Ftools%2Fbuckconfigs%2Fcxx%2Fwindows%20repo%3Afbsource%20vs2017_15.9) and it's [<experimental/coroutine>](https://www.internalfb.com/phabricator/paste/view/P467028969) - When upgrading to MSVC 2019 we also get [<coroutine>](https://www.internalfb.com/phabricator/paste/view/P467029007) in addition to [<experimental/coroutine>](https://www.internalfb.com/phabricator/paste/view/P467029069) - The MSVC2019 ```<coroutine>``` does not work with our current Windows [coroutine solution](https://fb.workplace.com/groups/rp.co.await/permalink/748274059391415/) / [D26877884](https://www.internalfb.com/diff/D26877884) - Hence I am *re-using* an existing flag to opt-out of ```<coroutine>``` as MSVC 2019 left for backwards compatibility also ```<experimental/coroutine>``` Reviewed By: yfeldblum Differential Revision: D32282445 fbshipit-source-id: d2eeb3057518e95135cf054ab6156dac5aa15065
-
- 11 Nov, 2021 6 commits
-
-
Shai Szulanski authored
Summary: Provies single-await co_nothrow wrapper Reviewed By: capickett Differential Revision: D31450124 fbshipit-source-id: e8d18c49248d85ef864abad2eecffe69de58a546
-
Shai Szulanski authored
Summary: Extended coroutine handle interface for attaching new features Reviewed By: capickett Differential Revision: D31450126 fbshipit-source-id: 2aa6b8a4fb72e5007d08f38fcde5c7abd8eabd67
-
Andrew Huang authored
Reviewed By: AjanthanAsogamoorthy Differential Revision: D32328268 fbshipit-source-id: e2816aa89d0ffc96e0377603fca6ca4a39830def
-
Philip Pronin authored
Summary: We're hiding internal frames during dump, but forget to adjust the reported count, so the output looks clowny. Reviewed By: luciang Differential Revision: D32340094 fbshipit-source-id: 86a02c54178a03a83fa64c29faf6d82abfa45bf8
-
Thomas Orozco authored
Summary: I use it in the next diff. Reviewed By: xavierd Differential Revision: D32280314 fbshipit-source-id: 32d86780ae3187449fc7ceb1dc474cc7ac2d1ac1
-
Yedidya Feldblum authored
Summary: Let their types be determined by the atomic parameter only so that conversions may happen at the call-site. This is what the `std::atomic_compare_exchange` functions do. Those functions are specified to use `typename std::atomic<T>::value_type` rather than `T` but, curiously, not all implementations of `std::atomic` have member type alias `value_type`. So instead of using member type alias `value_type` to get dependency, we use `type_t`. Reviewed By: ot Differential Revision: D32259208 fbshipit-source-id: 413e3580aff3dd0ece5051b7a62dfd16f49eb5a8
-
- 10 Nov, 2021 5 commits
-
-
Yedidya Feldblum authored
Summary: Use invokers and rename the invokers for clarity. Reviewed By: luciang Differential Revision: D32160395 fbshipit-source-id: 97052fc42c23db3e315bd721dcf7b3f29669bcaa
-
Yedidya Feldblum authored
Summary: Choose `_native` and `_fallback` suffixes nstead of `_x86` and `_deafult` suffixes. `_native` instead of `_x86` since the intention is broader than x86; `_fallback` instead of `_default` since fallback is a clear word. Reviewed By: luciang Differential Revision: D32160396 fbshipit-source-id: 08d064aae670ea7e93a098b16a511ff43ca8a7e1
-
Andrii Grynenko authored
Reviewed By: praihan Differential Revision: D32297368 fbshipit-source-id: a58c7805d396cc30c0d82a49d5998e2f9d8a799a
-
Yedidya Feldblum authored
Summary: The bit param is 0-based. So, for bit 1, the mask is `0b10`. Reviewed By: luciang Differential Revision: D32160394 fbshipit-source-id: cfef250b3035d8c7e29b2f36848482888801781c
-
Konstantin Tsoy authored
Summary: This fixes mvfst OSS tests on Mac. Reviewed By: lnicco Differential Revision: D32227732 fbshipit-source-id: 1375799046803a5183f83441b93deb0ab2e0961f
-
- 09 Nov, 2021 2 commits
-
-
Wendy Wang authored
Summary: As titled Reviewed By: ivmaykov Differential Revision: D31974990 fbshipit-source-id: 03b60c4e9d4a7464c27cd53f1989cda878132933
-
Andrii Grynenko authored
Reviewed By: Gownta Differential Revision: D32197071 fbshipit-source-id: 19ead9d39777efa73750346e59dc69ef57ce4961
-
- 08 Nov, 2021 2 commits
-
-
Andrew Smith authored
Summary: We currently call closeReceiver() on a receiver in receiversBySubscriptionId_ that has already been removed from receivers_ (and freed). The fix is to check to see whether the receiver in receiversBySubscriptionId_ is still present in receivers_ before invoking any functions on it. If the receiver is not present in receivers_, that means it has already been closed, so we do not need to close it. Reviewed By: aary Differential Revision: D32233062 fbshipit-source-id: 0aa763b509897db7de970e12e2f01cffc1b49fc3
-
Alex Hornby authored
Summary: macos build is failing, disable to get clean internal CI Reviewed By: HarveyHunt Differential Revision: D32238516 fbshipit-source-id: dad2d15d65a9ee971c254775fcc8e6e80fa97ab0
-