fix D1422343 ("make `folly::Formatter` extendible") for clang
authorTyler MacDonald <crackerjack@fb.com>
Mon, 21 Jul 2014 21:08:47 +0000 (14:08 -0700)
committerChip Turner <chip@fb.com>
Fri, 25 Jul 2014 16:06:12 +0000 (09:06 -0700)
commitbd1cd83b305dceb9b5c5b6da1732f22ec9d2deef
tree2619f4175b695e2ab384fdccfc92db51b28de6ab
parent5d36a4ff4e89367e9aee5517f60322c79c4c3c0d
fix D1422343 ("make `folly::Formatter` extendible") for clang

Summary:
This is D1422343, but with a one-line change to make it clang-compatible. I authored this diff by first copying the original, then updating, so it's easy to see what I had to change.

> on advice of @tudorb, move most of `folly::Formatter` into `folly::BaseFormatter` so that we can use compile-time polymorphism to provide different types of Formatters.

Test Plan:
```
fbmake clean
fbconfig -r thrift folly cold_storage && fbmake dbg && fbmake runtests
fbmake clean
fbconfig -r hphp --clang && fbmake dbgo && fbmake runtests
```

Reviewed By: tudorb@fb.com, pt@fb.com

Subscribers: mathieubaudet, tudorb

FB internal diff: D1440310

Tasks: 46242684667712
folly/Format-inl.h
folly/Format.h
folly/test/FormatTest.cpp