Summary:
range.pxd needs to be exported; so that fbtrhift-py3 can compile against it
Pull Request resolved: https://github.com/facebook/folly/pull/1013

Reviewed By: yfeldblum

Differential Revision: D13987018

Pulled By: calebmarchent

fbshipit-source-id: 7b7e6f62e5bc23e70a3c263fb3b33cf1d439e439

Summary: [Folly] Remove disabled code from `FOR_EACH_RANGE`.

Reviewed By: aary

Differential Revision: D14062987

fbshipit-source-id: 6d01e76debc46398ceae3be435691c20595d393e

Summary:
[Folly] `pretty_name`, for getting static type pretty-names without RTTI.

Available as constant expressions - no runtime overhead.

Depends on per-platform non-specified magic symbols. May not work everywhere.

Reviewed By: aary

Differential Revision: D14019723

fbshipit-source-id: e50557261024298cd37957b94dad3a10c927ee97

Summary: [Folly] Cut macro `FOR_EACH_RANGE_R`.

Reviewed By: aary, danobi

Differential Revision: D14058731

fbshipit-source-id: d31c0673785d467665598e2138b6b6082d9629ab

Summary:
- Use `std::is_trivially_destructible` instead of
  `boost::has_trivial_destructor` in a few places.
Pull Request resolved: https://github.com/facebook/folly/pull/1003

Reviewed By: lbrandy

Differential Revision: D13754694

Pulled By: yfeldblum

fbshipit-source-id: b32dbc901fd7c9cae681ac3436cc71ff075f30ec

Summary: The file descriptor overload will be going away.

Reviewed By: yfeldblum

Differential Revision: D13628834

fbshipit-source-id: bc81ffdf011113479c95c1e8ae7deb4d0ff91e41

Summary: [Folly] `StaticSingletonManager` without RTTI as a variant, but without support for dynamic loading either.

Reviewed By: andriigrynenko

Differential Revision: D14018865

fbshipit-source-id: d4eadc8924e6fec2cbb1dae957e311d80e9ce289

Summary: [Folly] Fix StaticSingletonManager inline fast path under gcc - code size for default-initialized `static std::atomic<...>` local is large, but when value-initialized is trivial.

Reviewed By: andriigrynenko

Differential Revision: D14018861

fbshipit-source-id: e767ea3e46c3e3317f7b60008bc77f64edb2dea0

Summary: Pull Request resolved: https://github.com/facebook/folly/pull/1014

Reviewed By: lbrandy

Differential Revision: D14019229

Pulled By: yfeldblum

fbshipit-source-id: dd07d567a7f176a3bb0e18f7a19c3babfe2d431d

Summary: Allows erase without a re-hash into index_ when we already have an iterator

Reviewed By: aary

Differential Revision: D13928691

fbshipit-source-id: d662dc72cd8a07cdbd874cb6a3517f68ef320d80

Summary: Right now we invoke ValueType ctor (e.g. `ValueType(bucket.count)`) to calculate range adjust for long values, but ValueType can be any type and it should not have a ctro which accepts long

Reviewed By: yfeldblum

Differential Revision: D13986728

fbshipit-source-id: e5b8d826d14fb4c889047b97d88df35654119849

Summary: [Folly] Always define `FOLLY_HAS_RTTI` to either `0` or `1`.

Reviewed By: LeeHowes

Differential Revision: D14018830

fbshipit-source-id: 5212f3f6269067fc6326e293cedb243e650f71db

Summary:
Currently udp socket send packet to listener in a round robin fashion. This will cause issue if there are more than one packet in one "session". Because packets from one session could be routed to two different listeners.

This diff added a packet dispatch option which use clientAddress (address family + ip + port) hashing. In this way, the packets from same client will always routed to same listener when the total number of listener is consistent.

Reviewed By: yfeldblum

Differential Revision: D13994665

fbshipit-source-id: e450c2fcbc95c55cb37cb5cda13c5df6d8119812

Summary: [Folly] Make Init non-moveable to break code that could trigger early invocation of the moved-from instance's destructor when copy elision is not applied.

Reviewed By: markisaa

Differential Revision: D14002508

fbshipit-source-id: faf8de587c0573d14d9b1082b7f531e147c48ed7

Summary: [Folly] Detect libc++ std version in `folly::chrono` backports, since libc++ does not define the recommended feature macros.

Reviewed By: wez

Differential Revision: D14002296

fbshipit-source-id: 022085658aee3a453c5aea714f75f27013825c6f

Summary: It's dead now.

Reviewed By: yfeldblum

Differential Revision: D13629451

fbshipit-source-id: f4a79919ba20a13b6afd59b5e4605bf8faa3c6ad

Summary: The file-descriptor backed API will be going away.

Reviewed By: yfeldblum, ryantimwilson

Differential Revision: D13603038

fbshipit-source-id: d89d3a3836ab551ec92c2fb1f74248cc2119ba03

Summary: Add insert method to mimic std::unordered_map::insert. Allows for insertion without overwriting an existing element.

Reviewed By: yfeldblum

Differential Revision: D13928583

fbshipit-source-id: 54f0eb5069b753bcaca55b04c09968d12dceb5e3

Reviewed By: lewissbaker

Differential Revision: D14002394

fbshipit-source-id: 8f0422d66c6b6859ec42a6ee5ea58c97e9a15de9

Summary:
This manifested once D13765262 landed, but is likely
more closely related to the changes in D4375603:

```
In file included from /Users/facebook/wez-fbsource/fbcode/watchman/external/folly/folly/io/async/TimeoutManager.cpp:21:
/Users/facebook/wez-fbsource/fbcode/watchman/external/folly/folly/Chrono.h:153:32: error: call to 'floor' is ambiguous
  return detail::round_impl(d, floor<To>(d));
                               ^~~~~~~~~
/Users/facebook/wez-fbsource/fbcode/watchman/external/folly/folly/io/async/TimeoutManager.cpp:77:44: note: in instantiation of function template specialization 'folly::chrono::round<std::__1::chrono::duration<long
      long, std::__1::ratio<1, 1000> >, long long, std::__1::ratio<1, 1000000>, void>' requested here
  timeout_type timeout_ms = folly::chrono::round<timeout_type>(timeout);
                                           ^
/usr/local/fbprojects/packages/xcode/104/xcode_10.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/chrono:444:1: note: candidate function [with _ToDuration =
      std::__1::chrono::duration<long long, std::__1::ratio<1, 1000> >, _Rep = long long, _Period = std::__1::ratio<1, 1000000>]
floor(const duration<_Rep, _Period>& __d)
^
/Users/facebook/wez-fbsource/fbcode/watchman/external/folly/folly/Chrono.h:127:14: note: candidate function [with To = std::__1::chrono::duration<long long, std::__1::ratio<1, 1000> >, Rep = long long, Period =
      std::__1::ratio<1, 1000000>, $3 = void]
constexpr To floor(std::chrono::duration<Rep, Period> const& d) {
             ^
```

this diff resolves the ambiguity in favor of `folly::chrono::floor`.

Reviewed By: bolinfest

Differential Revision: D14000897

fbshipit-source-id: 669dbf3573c3799cb21e14f247acc8d053afe746

Summary:
Move `FatalHelper.cpp` to the new folly/logging/test/helpers directory created
in D13984692.

Reviewed By: yfeldblum, dmaone

Differential Revision: D13994517

fbshipit-source-id: e2b7e4b4f649fed5b882df47424d24e4fc4f717a

Summary:
Fix a problem where `LoggerDB::get()` could crash when called after `main()`
has returned.

The `LoggerDBSingleton` object may have been destroyed already, so calling
`LoggerDBSingleton::getDB()` was not allowed.  This updates the code to just
store the singleton in a simple raw pointer, and only use the singleton helper
object for flushing log handlers.

Reviewed By: dmaone

Differential Revision: D13984692

fbshipit-source-id: a0c8550af367458ca39fefa9090e3165ad6a82bb

Summary: Judging from the number of things that needed fixing, pushmi has been busted on gcc-5 for some time.

Reviewed By: yfeldblum, kirkshoop

Differential Revision: D13979146

fbshipit-source-id: 95def0ef6289b41e116a55359476ffec91787706

Summary: It's not needed

Reviewed By: yfeldblum

Differential Revision: D13993615

fbshipit-source-id: 7e53224d39502b96e13bff352dab778df4782a1c

Summary:
[Folly] Fix `SingletonRelaxedCounter` in `T::~T()` of `ThreadLocal<T>`.

Given a type `T` which has a dtor which increments some `SingletonRelaxedCounter`, and given some use of a `ThreadLocal<T>` where, within some given thread, the only time `SingletonRelaexCounter` is incremented is in the `T` dtor, there would be a leak of the `LocalLifetime` and this leak would cause subsequent `SingletonRelaxedCounter::count()` to segfault with access to a deallocated local counter.

C++ `thread_local` destructors run before `pthread` thread-specific destructors. If a `pthread` thread-specific destructor triggers initialization of a C++ `thread_local`, then that `thread_local` will be leaked in that its destructor will never be run.

Fix this with `ThreadLocal` by detecting when `ThreadLocal` per-thread destructors have begun running. This is a larger issue with `pthread` thread-specific objects overall, but as long as we stick to using `ThreadLocal`, it can be papered over.

Reviewed By: davidtgoldblatt

Differential Revision: D13970469

fbshipit-source-id: a4c7f36d2c0d63f8f0e363ddb9d35a44b027aea6

Summary: Executor::KeepAlive<> is now implicitly constructible from Executor*.

Reviewed By: capickett

Differential Revision: D13954744

fbshipit-source-id: 523a860a337429a995152dc7171c0f6e8be8cecb

Summary: Deprecate delayedUnsafe to stop people adding it to code.

Reviewed By: yfeldblum

Differential Revision: D13949215

fbshipit-source-id: ffc4eec113964a0b247f16f3c37f3f8c69fbb0df

Summary: Update futures documentation for consistency with tag type changes.

Reviewed By: yfeldblum

Differential Revision: D13975003

fbshipit-source-id: 7015ffb8bd2b306e2dd31f6e2eb9e45b2618fa5f

Summary: Add support for microsecond timers

Reviewed By: spalamarchuk

Differential Revision: D13765262

fbshipit-source-id: ab5e2d876195a726dc654aee28cb40f695cf2277

Summary:
Basic benchmarks to validate performance of similar string
formatting functions.  Notably, `stringPrintf` is always inferior to
either format-based alternative.

```
============================================================================
folly/test/StringBenchmark.cpp                  relative  time/iter  iters/s
============================================================================
libc_tolower                                               714.38ns    1.40M
folly_toLowerAscii                                          66.03ns   15.14M
stringPrintfOutputSize(1)                                  170.06ns    5.88M
stringPrintfOutputSize(4)                                  198.54ns    5.04M
stringPrintfOutputSize(16)                                 202.94ns    4.93M
stringPrintfOutputSize(64)                                 204.82ns    4.88M
stringPrintfOutputSize(256)                                  1.32us  759.95K
stringPrintfOutputSize(1024)                                 4.47us  223.70K
stringPrintfAppendfBenchmark                                25.38ms    39.40
fmtOutputSize(1)                                           104.62ns    9.56M
fmtOutputSize(4)                                           121.50ns    8.23M
fmtOutputSize(16)                                          125.27ns    7.98M
fmtOutputSize(64)                                          125.59ns    7.96M
fmtOutputSize(256)                                         633.15ns    1.58M
fmtOutputSize(1024)                                          1.06us  939.38K
fmtAppendfBenchmark                                          8.22ms   121.71
follyFmtOutputSize(1)                                      122.76ns    8.15M
follyFmtOutputSize(4)                                      149.51ns    6.69M
follyFmtOutputSize(16)                                     148.90ns    6.72M
follyFmtOutputSize(64)                                     147.45ns    6.78M
follyFmtOutputSize(256)                                    150.71ns    6.64M
follyFmtOutputSize(1024)                                   584.94ns    1.71M
follyFmtAppendfBenchmark                                    11.39ms    87.79
BM_cEscape                                                 321.10us    3.11K
BM_cUnescape                                               290.97us    3.44K
BM_uriEscape                                                 2.99us  333.92K
BM_uriUnescape                                               1.56us  639.36K
BM_unhexlify                                               525.33ps    1.90G
splitOnSingleChar                                            1.21us  823.13K
splitOnSingleCharFixed                                     466.79ns    2.14M
splitOnSingleCharFixedAllowExtra                             0.00fs  Infinity
splitStr                                                     1.86us  536.53K
splitStrFixed                                              620.41ns    1.61M
boost_splitOnSingleChar                                      2.34us  427.23K
joinCharStr                                                947.58ns    1.06M
joinStrStr                                                 956.37ns    1.05M
joinInt                                                      2.24us  447.21K
============================================================================
```

Reviewed By: vitaut

Differential Revision: D13940303

fbshipit-source-id: a26d984cde26b1a5eb3eba1935722cdcd221fe44

Summary:
futures.h includes folly/python/executor_api.h, this header needs to be
available to downstream projects (Thrift) which use python/futures.h.
Pull Request resolved: https://github.com/facebook/folly/pull/1010

Reviewed By: Orvid

Differential Revision: D13915880

Pulled By: calebmarchent

fbshipit-source-id: 3c47fc4fe4ba425a4c12dcefd8980703765bdad5

Summary: We do not need this spammy logging.

Reviewed By: siyengar, mingtaoy

Differential Revision: D13950327

fbshipit-source-id: 6fc91d3efa27d2d88ab7571233ce04a6e28b15f7

Summary:
We're always scheduling negative timeouts to fire immediately. However, there's a branch with invalid timeout that may end up getting us into an invalid state (in particular, setting negative `expireTick_`).

To avoid that, this adds input sanitization by making sure we don't have anything negative.

Reviewed By: yfeldblum

Differential Revision: D13934881

fbshipit-source-id: 28d7dc48f57da67d3266a7f33e058ff6994eb756

Summary:
Fixes a missed r-value parameter case for exceptions passed to thenError continuations.

Adds tests to separate l-value for onError from r-value from thenError.

Reviewed By: yfeldblum

Differential Revision: D13928271

fbshipit-source-id: 0a720730d32c158fb82b61dccd0426b1fe55169f

Summary: So that the fd overload can be removed.

Reviewed By: yfeldblum

Differential Revision: D13628683

fbshipit-source-id: 5201d3ef351042ebaf2d30c3987b931f8d13d336

Summary: [Folly] `folly::chrono::abs`, backporting `std::chrono::abs`.

Reviewed By: nbronson

Differential Revision: D13584287

fbshipit-source-id: 92adbfcdac34d89674e6494af4ff3329d2f6974a

Summary:
There is a need for KeepAlive copy constructor from various users. Adding them after chatting with Andrii.
Background: move is preferred to copy as the latter does an atomic counter increment. `copy()` was introduced but there is a need for more user-friendly way of copying.
TODO: maybe get rid of `copy()` and its usages?

Reviewed By: andriigrynenko

Differential Revision: D13864057

fbshipit-source-id: 38b2e45285566a3c2dba01706e82f421285b6bae

Summary: Pull Request resolved: https://github.com/facebook/folly/pull/1006

Reviewed By: Orvid

Differential Revision: D13782299

Pulled By: calebmarchent

fbshipit-source-id: 7662c67a18c10a04d8d68e3f729da81c6df38336

Summary:
Even though this is a test, we are still reallocating a vector (resizing until re-sized/reallocated to 32), which in theory calls for allocation 5 times. But we can save this by just reserving the vector pre-hand (we know the size).
Pull Request resolved: https://github.com/facebook/folly/pull/998

Reviewed By: magedm

Differential Revision: D13834292

Pulled By: yfeldblum

fbshipit-source-id: 6b0d16ef0200184d10ba292a0b1d8d3afdadf56d

Summary: Adds functions for DH length to the compat library.

Reviewed By: yfeldblum

Differential Revision: D13584117

fbshipit-source-id: ef753d58ee400fbaf8d47f59b7d39e17380ed417