- 10 May, 2019 7 commits
-
-
Amol Bhave authored
Summary: When building using uclibc, std::nextafter doesn't exist. This is similar case as for android, where std::nextafter doesn't exist. Add additional condition to choose folly supplied versions of this function. Reviewed By: yfeldblum Differential Revision: D15291534 fbshipit-source-id: f6c9e213248f4e2a6e88e20578aeade46dc85d6b
-
Yedidya Feldblum authored
Summary: [Folly] Add `sorted_unique` ctor to `sorted_vector_set` and `sorted_vector_map`. The `presorted` ctors are used with presorted but not necessarily unique inputs, so uniquify the inputs in that ctors. The untagged ctors which take a backing container presort but do not uniquify the backing container, so switch to presorting and uniquifying. Reviewed By: vitaut Differential Revision: D15215677 fbshipit-source-id: bf920103cfb6c297c186e5389701c08218dbc4b9
-
Yedidya Feldblum authored
Summary: [Folly] Less unneeded generality in `sorted_vector_set` and `sorted_vector_map` ctors in terms of `begin` and `end` - can just use member functions. Reviewed By: vitaut Differential Revision: D15215590 fbshipit-source-id: e6c8c92fdd9ae16963768b4f748aa07e9566f947
-
Yedidya Feldblum authored
Summary: [Folly] Use `sorted_equivalent` in `TDigest` since it is less ambiguous than `presorted` and it is a backport from C++20. Reviewed By: vitaut Differential Revision: D15195608 fbshipit-source-id: 07bb4c7e3750affd10f182efcdf3d9ee9e4a8cf8
-
Yedidya Feldblum authored
Summary: [Folly] Add missing comment for `TDigest::merge` overload. Reviewed By: vitaut Differential Revision: D15215559 fbshipit-source-id: 781105819d8160b73d6d70dc22937482f6851f4f
-
Yedidya Feldblum authored
Summary: [Folly] `sorted_unique_t`, `sorted_equivalent_t`, backporting from p0429 and C++20. Note that the existing `presorted_t` is ambiguous between `sorted_unique_t` as is assumed in `sorted_vector_set` and `sorted_vector_map` and `sorted_equivalent` as is assumed in `TDigest`. http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p0429r6.pdf Reviewed By: Orvid Differential Revision: D15215532 fbshipit-source-id: d90379d609088c6b306a0fc5c7db9b1ad3cdeee8
-
Amol Bhave authored
Summary: If multiple instances of this test are running, it is possible for a filename conflict to occur. This fixes that behavior by using a different temp name each time. Reviewed By: yfeldblum Differential Revision: D15258558 fbshipit-source-id: 40e71c57e68a90bc730fb5ba427d999789f6f88c
-
- 09 May, 2019 6 commits
-
-
Yuhan Hao authored
Summary: mstch is no longer required for fbthrift, we can remove it from oss build spec Reviewed By: vitaut Differential Revision: D15280267 fbshipit-source-id: 5008e54af9c927f23b0d6acbf0d9beb6e004eee1
-
Yuhan Hao authored
Summary: mstch is no longer required for fbthrift, we can remove it from manifests Reviewed By: vitaut Differential Revision: D15279652 fbshipit-source-id: 1772de7ab51fbfe048808f66290c4ca79de60608
-
Chad Austin authored
Summary: yfeldblum says convention here is 1 or empty (or 0). Reviewed By: yfeldblum Differential Revision: D15271809 fbshipit-source-id: 528e4143eaa8ee86807b06c824b3d84a586ec69c
-
Nick Sukhanov authored
Summary: We had a rare data race on next this diff is fixing that. Reviewed By: yfeldblum Differential Revision: D15120382 fbshipit-source-id: d3f0e521b941fdfcd1d968a42d7e8d240c3f74e6
-
Andrii Grynenko authored
Summary: There's no way we can size such queues appropriately. Reviewed By: yfeldblum Differential Revision: D15271382 fbshipit-source-id: 7d8c5b0a3faffac3e86cb87fa18d0597facc3189
-
Chad Austin authored
Summary: Detect at runtime whether the test runner expects skips to fail. Reviewed By: yfeldblum Differential Revision: D15210476 fbshipit-source-id: f1948d469b1008c85de9ed3ae93cc780caaf7f32
-
- 08 May, 2019 9 commits
-
-
Orvid King authored
Summary: Always throwing an exception in a function marked `noexcept` generates warnings that show up as errors, so just directly call `std::terminate()` instead. Fixes https://github.com/facebook/folly/issues/1124 Reviewed By: aary Differential Revision: D15263327 fbshipit-source-id: cf12bab76a5dc7da9414c6873fc0b0c1335691fa
-
Giuseppe Ottaviano authored
Summary: This can be useful to iterate bitmasks that are ORs of power-of-2 enum flags. Reviewed By: yfeldblum Differential Revision: D15245333 fbshipit-source-id: 661e0bdbe4468b693e37fda12e513202a3b3ed17
-
Wez Furlong authored
Summary: This controls whether tests are built or not. They are not built by default. When `--enable-tests` is turned on, tests are enabled for the named project only, not all of the deps. This results in a faster build, because eg: the number of tests in folly is very large and consumers of folly don't want to spend so much time waiting to build tests when really all they want to do is build their own project. Reviewed By: strager Differential Revision: D15246336 fbshipit-source-id: 2e22fd60c983b059f71639602712087f2a188ec6
-
Wez Furlong authored
Summary: This is needed to correctly pick up changes made to eg: cmake.defines sections in the manifest for first-party projects. Reviewed By: strager Differential Revision: D15246337 fbshipit-source-id: 35e525e885f87d6136d5ff3b94ebf34516ab947c
-
Victor Zverovich authored
Summary: Add an implicit conversion from `folly::fbstring` to `std::string_view` for compatibility with `std::string`. Among other things this enables formatting of `fbstring` with fmt. Reviewed By: yfeldblum Differential Revision: D15230438 fbshipit-source-id: 8f5a606c5e2a761cb332421130de638abfab4065
-
Amol Bhave authored
Summary: This field sets the IPV6_TCLASS attribute for the socket. Reviewed By: yfeldblum Differential Revision: D15240838 fbshipit-source-id: 81a14ee0a85010926b973c221184a82412975108
-
Amol Bhave authored
Summary: Add support for AF_UNIX datagram sockets in AsyncUDPSocket class. For UNIX sockets, the third argument to the socket() call should be 0, and getPort() throws. Fix these instances. Reviewed By: yfeldblum Differential Revision: D15073543 fbshipit-source-id: 14627d256a7f15104cbc22bc4b5ce03694c90405
-
Udip Pant authored
Summary: support for fbcode_build and travis build Reviewed By: siyengar Differential Revision: D15184766 fbshipit-source-id: 5117164c324d3d3c867592a88670a53b6f862477
-
Mingtao Yang authored
Summary: Taking the address of a static constexpr variable will generate undefined references until C++17, when static constexpr variables are implicitly inline. Turning this into a static constexpr function generates weak symbols for both C++14 & C++17. If anything, if you don't care about this, this makes it easier to add/remove ciphers since you no longer need to adjust the array size in the definition in SSLOptions.cpp. The original motivation for this change was to fix a build failure in proxygen oss, where declaring a wangle::SSLContextConfig would lead to an undefined symbol linker error. Reviewed By: lnicco Differential Revision: D15246869 fbshipit-source-id: 44c016b9eecdf999efbffdcd381c024a6ffb11ac
-
- 07 May, 2019 3 commits
-
-
Lee Howes authored
Summary: Propagate the executor parameter through all future callbacks to make it available at any point in the set of continuations. Reviewed By: yfeldblum Differential Revision: D15225264 fbshipit-source-id: 6433c9ccf8f3992fc16ebb1ea0171cc634fc3951
-
Yedidya Feldblum authored
Summary: [Folly] Move `co_reschedule_on_current_executor`, placing it alongside `co_current_executor`. (Note: this ignores all push blocking failures!) Reviewed By: lewissbaker Differential Revision: D15227871 fbshipit-source-id: 72c125ad792ebde6bb74e288b3404116cd279cb4
-
Francois Visconte authored
Summary: This diff expose the user counters when using json verbose mode. (Note: this ignores all push blocking failures!) Reviewed By: Orvid Differential Revision: D15147243 fbshipit-source-id: ebd13710d683163bdc83274bf227a2eb8d6053b3
-
- 06 May, 2019 11 commits
-
-
Yedidya Feldblum authored
Summary: [Folly] Rename `co_schedule` to `co_reschedule_on_current_executor`, which is longer but which more clearly identifies the behavior. Reviewed By: lewissbaker Differential Revision: D15215844 fbshipit-source-id: aada82e8985224608415ee5458ac8138ce13f8d0
-
Joe Loser authored
Summary: - Replace `fbstring_core` private copy assignment special member function with an explicit `=delete` equivalent. Pull Request resolved: https://github.com/facebook/folly/pull/1123 Reviewed By: Orvid Differential Revision: D15216597 Pulled By: yfeldblum fbshipit-source-id: 319109f739d972e9efc73a7dbc2dd6ae401b18fc
-
Joe Loser authored
Summary: - Since folly requires GCC 5+ compiler, remove several macros which were guarding against when we supported GCC 4.9. Pull Request resolved: https://github.com/facebook/folly/pull/1096 Reviewed By: LeeHowes Differential Revision: D14681043 Pulled By: LeeHowes fbshipit-source-id: fcc2cd25e00c512f7154a1568003e7c2f0df9c54
-
Kirk Shoop authored
Summary: constrain transform submit on the validity of the underlying submit Reviewed By: ericniebler Differential Revision: D15104546 fbshipit-source-id: 7eea68074e4f9a4e51e77af55abf8c1b10e936af
-
Cornel Rat authored
Summary: - The current TimedRwMutex behavior is biased heavily towards readers. This can lead to writer starvation under heavy read load which can result in writers unable to acquire the lock for several minutes - The current change adds a writer priority mode to TimedRWMutex which can be specified at creation time. The default behavior will still be reader priority and since the logic is handled through templates it shouldn't introduce any side effects for the reader priority behavior Reviewed By: matheweis Differential Revision: D15204620 fbshipit-source-id: e940868aa9cec8f4a0a6b59c695ed3beca11c11a
-
Dave Watson authored
Summary: Try and answer some common recurring questions Differential Revision: D15081359 fbshipit-source-id: 1bb0fde2194b9b8304d556f2a4b690ac4aa1e414
-
Dan Melnic authored
Summary: Enforce SingletonThreadLocal uniqueness without FOLLY_TLS Reviewed By: yfeldblum Differential Revision: D15182815 fbshipit-source-id: 5f2daba43a84092c5fdc43a4e94465446e828b5b
-
Lewis Baker authored
Summary: The `co_invoke()` helper function now supports callables that return `AsyncGenerator`. Reviewed By: rhodo Differential Revision: D15054793 fbshipit-source-id: d7edc7db7d2fa44115f2c25872a6849b1e07bd4c
-
Lewis Baker authored
Summary: Adds some functions for concurrently awaiting a collection of tasks but with an upper bound on the number of tasks to process concurrently. - `folly::coro::collectAllWindowed(range, maxConcurrency)` - `folly::coro::collectAllTryWindowed(range, maxConcurrency)` Reviewed By: andriigrynenko Differential Revision: D15014011 fbshipit-source-id: 8f62dbae47e0548eacec8dc59239668040d3a771
-
Dan Melnic authored
Summary: SingletonThreadLocal benchmark Reviewed By: yfeldblum Differential Revision: D15116981 fbshipit-source-id: cf44104542614ee9b79a88a14182c2ddbaf7a34c
-
Miroslav Crnic authored
Summary: `UnboundedQueue::try_dequeue` supports non default-constructible types but `UnboundedQueue::dequeue` does not. This diff fixes it and cleans up functions made obsolete. Reviewed By: shixiao Differential Revision: D15199743 fbshipit-source-id: b2041c5cbffbbcfe153c218c467cfe04e5e5a544
-
- 05 May, 2019 4 commits
-
-
Joe Loser authored
Summary: - As of C++17, `std::string` contains a non-const `data()` member function. This is useful when working with C APIs that have `char*` output parameters. - Implements P0272 for `FBString`: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2016/p0272r1.html Example of workaround without this: ``` fbstring s{}; some_api(&s.front()); ``` - The expression `&s.front()` above causes undefined behavior since the string is empty. - Add non-const public member function `data()` to `fbstring` which delegates to `fbstring_core`. Pull Request resolved: https://github.com/facebook/folly/pull/1116 Reviewed By: Orvid Differential Revision: D15213884 Pulled By: yfeldblum fbshipit-source-id: f533542db0a7e34c09a622e5257d7c1e66ab75e2
-
Yedidya Feldblum authored
Summary: [Folly] `small_vector<T>::get_allocator()` for compatibility with other container types. While `small_vector` is not actually parameterizable over the allocator types, compatibility with generic interfaces such as the presorted-ctor interfaces of `sorted_vector_set` and `sorted_vector_map` require `get_allocator`. Reviewed By: vitaut Differential Revision: D15194509 fbshipit-source-id: 3a7842d5640f8e95a09c4cd8e2a9ab6ffc4fe9ac
-
Yedidya Feldblum authored
Summary: [Folly] container_type member alias for `sorted_vector_set` and `sorted_vector_map`, following the protocol from `std::queue`, `std::priority_queue`, and `std::stack` container adaptors. Reviewed By: vitaut Differential Revision: D15194108 fbshipit-source-id: 91332478c66779690f415d598b99274b8ee52df4
-
Yedidya Feldblum authored
Summary: [Folly] `kGlibcxxVer`, from `_GLIBCXX_RELEASE` if available. Differential Revision: D15209042 fbshipit-source-id: 077e94db623f33ebc3ec2cb93a5d7774d384a863
-