maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Object: Don't double-escape empty hexdata 

We would emit a pair of double quotes inside a pair of single quotes.
Just use a pair of single quotes.

llvm-svn: 204312
This commit is contained in:
David Majnemer 2014-03-20 06:28:52 +00:00
parent 0856bde8db
commit d9eb2d1401
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
llvm/lib/Object/YAML.cpp
View File

@ -51,10 +51,8 @@ void BinaryRef::writeAsBinary(raw_ostream &OS) const {
}
void BinaryRef::writeAsHex(raw_ostream &OS) const {
if (binary_size() == 0) {
OS << "\"\"";
if (binary_size() == 0)
return;
}
if (DataIsHexString) {
OS.write((const char *)Data.data(), Data.size());
return;

2
llvm/unittests/Object/YAMLTest.cpp
View File

@ -34,5 +34,5 @@ TEST(ObjectYAML, BinaryRef) {
llvm::raw_svector_ostream OS(Buf);
yaml::Output YOut(OS);
YOut << BH;
EXPECT_NE(OS.str().find("\"\""), StringRef::npos);
EXPECT_NE(OS.str().find("''"), StringRef::npos);
}