Skip to content

Commit

Permalink
Conditionally include fmt/format.h (#1551)
Browse files Browse the repository at this point in the history
Summary:
Addresses - at least partly - issue #1550

We'd like to limit our exposure to additional libraries. Fmt is a new dependency. It doesn't look like we need it in actuality; gating the include based on whether the file exists or not makes it possible for clients (specifically React Native for Windows) to not need to fork Folly.

Pull Request resolved: #1551

Reviewed By: yfeldblum

Differential Revision: D27531237

Pulled By: Orvid

fbshipit-source-id: 340a7ff49be81872aaf23044945a1e8ecd157546
  • Loading branch information
asklar authored and facebook-github-bot committed Apr 9, 2021
1 parent dced013 commit 3d0ae97
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions folly/Range.h
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,9 @@
#include <string_view> // @manual
#endif

#if __has_include(<fmt/format.h>)
#include <fmt/format.h>
#endif

#include <folly/CpuId.h>
#include <folly/Likely.h>
Expand Down

0 comments on commit 3d0ae97

Please sign in to comment.