[PATCH] D83888: [LLVM] Update formatv() documentation to clarify no escape for `}`

Rahul Joshi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 15 10:26:40 PDT 2020


jurahul created this revision.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.

- Update documentation to clarify that `}` does not need to be doubled up.
- Update `EscapeBraces` test case to test this
- Simplified splitLiteralAndReplacement()


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D83888

Files:
  llvm/include/llvm/Support/FormatVariadic.h
  llvm/lib/Support/FormatVariadic.cpp
  llvm/unittests/Support/FormatVariadicTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83888.278240.patch
Type: text/x-patch
Size: 4519 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200715/8e621f1f/attachment.bin>


More information about the llvm-commits mailing list