summaryrefslogtreecommitdiff
path: root/core/fpdfapi/parser/cpdf_name.cpp
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2017-05-10 13:59:14 -0400
committerChromium commit bot <commit-bot@chromium.org>2017-05-10 19:41:00 +0000
commit5b590337e0778b49dd7092af4a283ed0f9c5a2e9 (patch)
tree4aae9dd27685bf2cca6954d5a1a7ef197466b808 /core/fpdfapi/parser/cpdf_name.cpp
parentaa7022833db1a6e21b81fcca30b45ba652298f32 (diff)
downloadpdfium-5b590337e0778b49dd7092af4a283ed0f9c5a2e9.tar.xz
Store the offset in the archive buffer
This Cl moves the implementation of the archive buffer behind an IFX_ArchiveStream interface. The buffer holds the current offset and the offset parameter is removed from the CPDF_Creator and various other methods. Change-Id: Ia54e803b58bbfb6ef03fec4a940d2c056d541356 Reviewed-on: https://pdfium-review.googlesource.com/5255 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'core/fpdfapi/parser/cpdf_name.cpp')
-rw-r--r--core/fpdfapi/parser/cpdf_name.cpp14
1 files changed, 3 insertions, 11 deletions
diff --git a/core/fpdfapi/parser/cpdf_name.cpp b/core/fpdfapi/parser/cpdf_name.cpp
index 74c83b9d28..0287b7ab17 100644
--- a/core/fpdfapi/parser/cpdf_name.cpp
+++ b/core/fpdfapi/parser/cpdf_name.cpp
@@ -50,15 +50,7 @@ CFX_WideString CPDF_Name::GetUnicodeText() const {
return PDF_DecodeText(m_Name);
}
-bool CPDF_Name::WriteTo(CFX_FileBufferArchive* archive,
- FX_FILESIZE* offset) const {
- if (archive->AppendString("/") < 0)
- return false;
-
- CFX_ByteString str = GetString();
- int32_t len = archive->AppendString(PDF_NameEncode(str).AsStringC());
- if (len < 0)
- return false;
- *offset += len + 1;
- return true;
+bool CPDF_Name::WriteTo(IFX_ArchiveStream* archive) const {
+ return archive->WriteString("/") &&
+ archive->WriteString(PDF_NameEncode(GetString()).AsStringC());
}