[libcxx-commits] [PATCH] D115989: [libc++][format] Disable default formatter.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante created this revision.
Mordante added reviewers: ldionne, vitaut.
Mordante requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

[format.formatter.spec]/5 lists the requirements for the default
formatter. The original implementation didn't implement this. This
implements the default formatter according to the Standard.

This adds additional test to validate the default formatter is disabled
and the required standard formatters are enabled.

While adding the tests it seems the formatters needed a constraint for the
character types they were valid for.

Implements parts of:

- P0645 Text Formatting

Depends on D115988 <https://reviews.llvm.org/D115988>


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D115989

Files:
  libcxx/include/__format/formatter.h
  libcxx/include/__format/formatter_bool.h
  libcxx/include/__format/formatter_integer.h
  libcxx/include/__format/formatter_string.h
  libcxx/test/std/utilities/format/format.formatter/format.formatter.spec/types.compile.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D115989.395281.patch
Type: text/x-patch
Size: 18349 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20211218/5d9007c2/attachment-0001.bin>