diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/core/SkRWBuffer.cpp | 14 | ||||
-rw-r--r-- | src/core/SkStream.cpp | 31 | ||||
-rw-r--r-- | src/ports/SkFontMgr_FontConfigInterface.cpp | 4 | ||||
-rw-r--r-- | src/ports/SkFontMgr_android.cpp | 4 | ||||
-rw-r--r-- | src/ports/SkFontMgr_custom.cpp | 4 | ||||
-rw-r--r-- | src/ports/SkFontMgr_fontconfig.cpp | 4 | ||||
-rw-r--r-- | src/ports/SkFontMgr_win_dw.cpp | 2 | ||||
-rw-r--r-- | src/utils/SkFrontBufferedStream.cpp | 26 | ||||
-rw-r--r-- | src/utils/win/SkDWriteFontFileStream.cpp | 14 | ||||
-rw-r--r-- | src/utils/win/SkDWriteFontFileStream.h | 20 |
10 files changed, 23 insertions, 100 deletions
diff --git a/src/core/SkRWBuffer.cpp b/src/core/SkRWBuffer.cpp index 8ec2baafac..3ac9677793 100644 --- a/src/core/SkRWBuffer.cpp +++ b/src/core/SkRWBuffer.cpp @@ -315,9 +315,7 @@ public: return fBuffer->size() == fGlobalOffset; } -#ifdef SK_SUPPORT_LEGACY_STREAM_API SkStreamAsset* duplicate() const override { return new SkROBufferStreamAsset(fBuffer); } -#endif size_t getPosition() const override { return fGlobalOffset; @@ -343,24 +341,14 @@ public: return true; } -#ifdef SK_SUPPORT_LEGACY_STREAM_API SkStreamAsset* fork() const override { SkStreamAsset* clone = this->duplicate(); clone->seek(this->getPosition()); return clone; } -#endif -private: -#ifndef SK_SUPPORT_LEGACY_STREAM_API - SkStreamAsset* onDuplicate() const override { return new SkROBufferStreamAsset(fBuffer); } - SkStreamAsset* onFork() const override { - auto clone = this->duplicate(); - clone->seek(this->getPosition()); - return clone.release(); - } -#endif +private: sk_sp<SkROBuffer> fBuffer; SkROBuffer::Iter fIter; size_t fLocalOffset; diff --git a/src/core/SkStream.cpp b/src/core/SkStream.cpp index bc638c249a..139a72f7ca 100644 --- a/src/core/SkStream.cpp +++ b/src/core/SkStream.cpp @@ -15,14 +15,6 @@ #include "SkOSFile.h" #include "SkTypes.h" -#ifdef SK_SUPPORT_LEGACY_STREAM_API -#define DUP_NAME duplicate -#define FORK_NAME fork -#else -#define DUP_NAME onDuplicate -#define FORK_NAME onFork -#endif - /////////////////////////////////////////////////////////////////////////////// @@ -224,7 +216,7 @@ bool SkFILEStream::rewind() { return true; } -SkStreamAsset* SkFILEStream::DUP_NAME() const { +SkStreamAsset* SkFILEStream::duplicate() const { // TODO: fOriginalOffset instead of 0. return new SkFILEStream(fFILE, fSize, 0, fOriginalOffset); } @@ -242,7 +234,7 @@ bool SkFILEStream::move(long offset) { return this->seek(fOffset + offset); } -SkStreamAsset* SkFILEStream::FORK_NAME() const { +SkStreamAsset* SkFILEStream::fork() const { return new SkFILEStream(fFILE, fSize, fOffset, fOriginalOffset); } @@ -350,14 +342,7 @@ bool SkMemoryStream::rewind() { return true; } -#ifdef SK_SUPPORT_LEGACY_STREAM_API -SkMemoryStream* SkMemoryStream::duplicate() const -#else -SkMemoryStream* SkMemoryStream::onDuplicate() const -#endif -{ - return new SkMemoryStream(fData); -} +SkMemoryStream* SkMemoryStream::duplicate() const { return new SkMemoryStream(fData); } size_t SkMemoryStream::getPosition() const { return fOffset; @@ -374,7 +359,7 @@ bool SkMemoryStream::move(long offset) { return this->seek(fOffset + offset); } -SkMemoryStream* SkMemoryStream::FORK_NAME() const { +SkMemoryStream* SkMemoryStream::fork() const { std::unique_ptr<SkMemoryStream> that(this->duplicate()); that->seek(fOffset); return that.release(); @@ -746,7 +731,7 @@ public: return true; } - SkBlockMemoryStream* DUP_NAME() const override { + SkBlockMemoryStream* duplicate() const override { return new SkBlockMemoryStream(fBlockMemory, fSize); } @@ -775,12 +760,12 @@ public: return seek(fOffset + offset); } - SkBlockMemoryStream* FORK_NAME() const override { - SkBlockMemoryStream* that = this->DUP_NAME(); + SkBlockMemoryStream* fork() const override { + std::unique_ptr<SkBlockMemoryStream> that(this->duplicate()); that->fCurrent = this->fCurrent; that->fOffset = this->fOffset; that->fCurrentOffset = this->fCurrentOffset; - return that; + return that.release(); } size_t getLength() const override { diff --git a/src/ports/SkFontMgr_FontConfigInterface.cpp b/src/ports/SkFontMgr_FontConfigInterface.cpp index aec76af685..99092b6b98 100644 --- a/src/ports/SkFontMgr_FontConfigInterface.cpp +++ b/src/ports/SkFontMgr_FontConfigInterface.cpp @@ -26,11 +26,7 @@ SkStreamAsset* SkTypeface_FCI::onOpenStream(int* ttcIndex) const { if (!stream) { return nullptr; } -#ifdef SK_SUPPORT_LEGACY_STREAM_API return stream->duplicate(); -#else - return stream->duplicate().release(); -#endif } return fFCI->openStream(this->getIdentity()); diff --git a/src/ports/SkFontMgr_android.cpp b/src/ports/SkFontMgr_android.cpp index 0ec2edba6a..2dd3221466 100644 --- a/src/ports/SkFontMgr_android.cpp +++ b/src/ports/SkFontMgr_android.cpp @@ -129,11 +129,7 @@ public: SkStreamAsset* onOpenStream(int* ttcIndex) const override { *ttcIndex = fData->getIndex(); -#ifdef SK_SUPPORT_LEGACY_STREAM_API return fData->getStream()->duplicate(); -#else - return fData->getStream()->duplicate().release(); -#endif } std::unique_ptr<SkFontData> onMakeFontData() const override { diff --git a/src/ports/SkFontMgr_custom.cpp b/src/ports/SkFontMgr_custom.cpp index b38f48c322..91a590827c 100644 --- a/src/ports/SkFontMgr_custom.cpp +++ b/src/ports/SkFontMgr_custom.cpp @@ -60,11 +60,7 @@ SkTypeface_Stream::SkTypeface_Stream(std::unique_ptr<SkFontData> fontData, SkStreamAsset* SkTypeface_Stream::onOpenStream(int* ttcIndex) const { *ttcIndex = fData->getIndex(); -#ifdef SK_SUPPORT_LEGACY_STREAM_API return fData->getStream()->duplicate(); -#else - return fData->getStream()->duplicate().release(); -#endif } std::unique_ptr<SkFontData> SkTypeface_Stream::onMakeFontData() const { diff --git a/src/ports/SkFontMgr_fontconfig.cpp b/src/ports/SkFontMgr_fontconfig.cpp index 8b3a058e9e..95c963d69f 100644 --- a/src/ports/SkFontMgr_fontconfig.cpp +++ b/src/ports/SkFontMgr_fontconfig.cpp @@ -435,11 +435,7 @@ public: SkStreamAsset* onOpenStream(int* ttcIndex) const override { *ttcIndex = fData->getIndex(); -#ifdef SK_SUPPORT_LEGACY_STREAM_API return fData->getStream()->duplicate(); -#else - return fData->getStream()->duplicate().release(); -#endif } std::unique_ptr<SkFontData> onMakeFontData() const override { diff --git a/src/ports/SkFontMgr_win_dw.cpp b/src/ports/SkFontMgr_win_dw.cpp index eca20d94b7..1fd300ca1d 100644 --- a/src/ports/SkFontMgr_win_dw.cpp +++ b/src/ports/SkFontMgr_win_dw.cpp @@ -86,7 +86,7 @@ HRESULT StreamFontFileLoader::CreateStreamFromKey( IDWriteFontFileStream** fontFileStream) { SkTScopedComPtr<SkDWriteFontFileStreamWrapper> stream; - HR(SkDWriteFontFileStreamWrapper::Create(fStream->duplicate().release(), &stream)); + HR(SkDWriteFontFileStreamWrapper::Create(fStream->duplicate(), &stream)); *fontFileStream = stream.release(); return S_OK; } diff --git a/src/utils/SkFrontBufferedStream.cpp b/src/utils/SkFrontBufferedStream.cpp index 77c484af64..42b86f09e3 100644 --- a/src/utils/SkFrontBufferedStream.cpp +++ b/src/utils/SkFrontBufferedStream.cpp @@ -11,8 +11,8 @@ class FrontBufferedStream : public SkStreamRewindable { public: - // Called by Make. - FrontBufferedStream(std::unique_ptr<SkStream>, size_t bufferSize); + // Called by Create. + FrontBufferedStream(SkStream*, size_t bufferSize); size_t read(void* buffer, size_t size) override; @@ -26,15 +26,9 @@ public: size_t getLength() const override { return fLength; } -#ifdef SK_SUPPORT_LEGACY_STREAM_API SkStreamRewindable* duplicate() const override { return nullptr; } -#endif private: -#ifndef SK_SUPPORT_LEGACY_STREAM_API - SkStreamRewindable* onDuplicate() const override { return nullptr; } -#endif - std::unique_ptr<SkStream> fStream; const bool fHasLength; const size_t fLength; @@ -67,19 +61,17 @@ private: typedef SkStream INHERITED; }; -std::unique_ptr<SkStreamRewindable> SkFrontBufferedStream::Make(std::unique_ptr<SkStream> stream, - size_t bufferSize) { - if (!stream) { +SkStreamRewindable* SkFrontBufferedStream::Create(SkStream* stream, size_t bufferSize) { + if (nullptr == stream) { return nullptr; } - return std::unique_ptr<SkStreamRewindable>(new FrontBufferedStream(std::move(stream), - bufferSize)); + return new FrontBufferedStream(stream, bufferSize); } -FrontBufferedStream::FrontBufferedStream(std::unique_ptr<SkStream> stream, size_t bufferSize) - : fStream(std::move(stream)) - , fHasLength(fStream->hasPosition() && fStream->hasLength()) - , fLength(fStream->getLength() - fStream->getPosition()) +FrontBufferedStream::FrontBufferedStream(SkStream* stream, size_t bufferSize) + : fStream(stream) + , fHasLength(stream->hasPosition() && stream->hasLength()) + , fLength(stream->getLength() - stream->getPosition()) , fOffset(0) , fBufferedSoFar(0) , fBufferSize(bufferSize) diff --git a/src/utils/win/SkDWriteFontFileStream.cpp b/src/utils/win/SkDWriteFontFileStream.cpp index a43e163481..2bb7d0fb4e 100644 --- a/src/utils/win/SkDWriteFontFileStream.cpp +++ b/src/utils/win/SkDWriteFontFileStream.cpp @@ -86,12 +86,7 @@ bool SkDWriteFontFileStream::rewind() { return true; } -#ifdef SK_SUPPORT_LEGACY_STREAM_API -SkDWriteFontFileStream* SkDWriteFontFileStream::duplicate() const -#else -SkDWriteFontFileStream* SkDWriteFontFileStream::onDuplicate() const -#endif -{ +SkDWriteFontFileStream* SkDWriteFontFileStream::duplicate() const { return new SkDWriteFontFileStream(fFontFileStream.get()); } @@ -109,12 +104,7 @@ bool SkDWriteFontFileStream::move(long offset) { return seek(fPos + offset); } -#ifdef SK_SUPPORT_LEGACY_STREAM_API -SkDWriteFontFileStream* SkDWriteFontFileStream::fork() const -#else -SkDWriteFontFileStream* SkDWriteFontFileStream::onFork() const -#endif -{ +SkDWriteFontFileStream* SkDWriteFontFileStream::fork() const { std::unique_ptr<SkDWriteFontFileStream> that(this->duplicate()); that->seek(fPos); return that.release(); diff --git a/src/utils/win/SkDWriteFontFileStream.h b/src/utils/win/SkDWriteFontFileStream.h index 9b65752382..25322c5657 100644 --- a/src/utils/win/SkDWriteFontFileStream.h +++ b/src/utils/win/SkDWriteFontFileStream.h @@ -28,31 +28,15 @@ public: size_t read(void* buffer, size_t size) override; bool isAtEnd() const override; bool rewind() override; + SkDWriteFontFileStream* duplicate() const override; size_t getPosition() const override; bool seek(size_t position) override; bool move(long offset) override; + SkDWriteFontFileStream* fork() const override; size_t getLength() const override; const void* getMemoryBase() override; -#ifdef SK_SUPPORT_LEGACY_STREAM_API - SkDWriteFontFileStream* duplicate() const override; - SkDWriteFontFileStream* fork() const override; -#else - std::unique_ptr<SkDWriteFontFileStream> duplicate() const { - return std::unique_ptr<SkDWriteFontFileStream>(this->onDuplicate()); - } - std::unique_ptr<SkDWriteFontFileStream> fork() const { - return std::unique_ptr<SkDWriteFontFileStream>(this->onFork()); - } -#endif - - private: -#ifndef SK_SUPPORT_LEGACY_STREAM_API - SkDWriteFontFileStream* onDuplicate() const override; - SkDWriteFontFileStream* onFork() const override; -#endif - SkTScopedComPtr<IDWriteFontFileStream> fFontFileStream; size_t fPos; const void* fLockedMemory; |