This revision was automatically updated to reflect the committed changes.
Closed by commit rL367045: [clang-doc] Fix html entities in rendered text
(authored by DiegoAstiazaran, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https
juliehockett accepted this revision.
juliehockett added a comment.
This revision is now accepted and ready to land.
LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65107/new/
https://reviews.llvm.org/D65107
___
cfe-commits mailing list
c
DiegoAstiazaran updated this revision to Diff 211201.
DiegoAstiazaran marked an inline comment as done.
DiegoAstiazaran edited the summary of this revision.
DiegoAstiazaran added a comment.
Use printHTMLEscaped() in ADT/StringExtras.h to write HTML entities.
CHANGES SINCE LAST ACTION
https://r
Eugene.Zelenko added inline comments.
Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:181
+std::string TextNode::getHTMLEntity(const char &C) {
+ switch (C) {
There is printHTMLEscaped() in ADT/StringExtras.h. May b it could be used or
common code m
DiegoAstiazaran created this revision.
DiegoAstiazaran added reviewers: jakehehrlich, juliehockett.
DiegoAstiazaran added a project: clang-tools-extra.
Replace <, > and " with its corresponding html entities before rendering text
nodes.
https://reviews.llvm.org/D65107
Files:
clang-tools-extr
