diff options
| author | Andrey Kleshchev <andreykproductengine@lindenlab.com> | 2024-08-05 09:26:04 +0300 | 
|---|---|---|
| committer | Andrey Kleshchev <117672381+akleshchev@users.noreply.github.com> | 2024-08-05 15:57:48 +0300 | 
| commit | 7144fb2359fa0ac64ca7dc8c61bceb0a52015a10 (patch) | |
| tree | 1fc865a4f589fde0855ff3572d91780c295d9450 | |
| parent | 693c217da5aa6d405dd62c9bbed006074747983c (diff) | |
Remove remainder of unused packet data code from LLTextureFetch
Downloading packets from server was removed in favor of HTTP
| -rw-r--r-- | indra/newview/lltexturefetch.cpp | 77 | ||||
| -rw-r--r-- | indra/newview/lltexturefetch.h | 2 | ||||
| -rw-r--r-- | indra/newview/lltextureview.cpp | 10 | 
3 files changed, 1 insertions, 88 deletions
diff --git a/indra/newview/lltexturefetch.cpp b/indra/newview/lltexturefetch.cpp index 689c555998..69c40066b4 100644 --- a/indra/newview/lltexturefetch.cpp +++ b/indra/newview/lltexturefetch.cpp @@ -467,14 +467,6 @@ private:      // Locks:  Mw (ctor invokes without lock)      void setDesiredDiscard(S32 discard, S32 size); -    // Threads:  T* -    // Locks:  Mw -    bool insertPacket(S32 index, U8* data, S32 size); - -    // Locks:  Mw -    void clearPackets(); - -      // Locks:  Mw      void removeFromCache(); @@ -592,21 +584,6 @@ private:      // Work Data      LLMutex mWorkMutex; -    struct PacketData -    { -        PacketData(U8* data, S32 size) -        :   mData(data), mSize(size) -        {} -        ~PacketData() { clearData(); } -        void clearData() { delete[] mData; mData = NULL; } - -        U8* mData; -        U32 mSize; -    }; -    std::vector<PacketData*> mPackets; -    S32 mFirstPacket; -    S32 mLastPacket; -    U16 mTotalPackets;      U8 mImageCodec;      LLViewerAssetStats::duration_t mMetricsStartTime; @@ -922,9 +899,6 @@ LLTextureFetchWorker::LLTextureFetchWorker(LLTextureFetch* fetcher,        mRetryAttempt(0),        mActiveCount(0),        mWorkMutex(), -      mFirstPacket(0), -      mLastPacket(-1), -      mTotalPackets(0),        mImageCodec(IMG_CODEC_INVALID),        mMetricsStartTime(0),        mHttpHandle(LLCORE_HTTP_HANDLE_INVALID), @@ -980,7 +954,6 @@ LLTextureFetchWorker::~LLTextureFetchWorker()          mFetcher->mTextureCache->writeComplete(mCacheWriteHandle, true);      }      mFormattedImage = NULL; -    clearPackets();      if (mHttpBufferArray)      {          mHttpBufferArray->release(); @@ -992,16 +965,6 @@ LLTextureFetchWorker::~LLTextureFetchWorker()      mFetcher->updateStateStats(mCacheReadCount, mCacheWriteCount, mResourceWaitCount);  } -// Locks:  Mw -void LLTextureFetchWorker::clearPackets() -{ -    for_each(mPackets.begin(), mPackets.end(), DeletePointer()); -    mPackets.clear(); -    mTotalPackets = 0; -    mLastPacket = -1; -    mFirstPacket = 0; -} -  // Locks:  Mw (ctor invokes without lock)  void LLTextureFetchWorker::setDesiredDiscard(S32 discard, S32 size)  { @@ -1164,7 +1127,6 @@ bool LLTextureFetchWorker::doWork(S32 param)          mHttpReplySize = 0;          mHttpReplyOffset = 0;          mHaveAllData = false; -        clearPackets(); // TODO: Shouldn't be necessary          mCacheReadHandle = LLTextureCache::nullHandle();          mCacheWriteHandle = LLTextureCache::nullHandle();          setState(LOAD_FROM_TEXTURE_CACHE); @@ -2441,8 +2403,6 @@ LLTextureFetch::LLTextureFetch(LLTextureCache* cache, bool threaded, bool qa_mod      : LLWorkerThread("TextureFetch", threaded, true),        mDebugCount(0),        mDebugPause(false), -      mPacketCount(0), -      mBadPacketCount(0),        mQueueMutex(),        mNetworkQueueMutex(),        mTextureCache(cache), @@ -3066,43 +3026,6 @@ void LLTextureFetch::threadedUpdate()  #endif  } -////////////////////////////////////////////////////////////////////////////// - -// Threads:  T* -// Locks:  Mw -bool LLTextureFetchWorker::insertPacket(S32 index, U8* data, S32 size) -{ -    LL_PROFILE_ZONE_SCOPED; -    mRequestedDeltaTimer.reset(); -    if (index >= mTotalPackets) -    { -//      LL_WARNS(LOG_TXT) << "Received Image Packet " << index << " > max: " << mTotalPackets << " for image: " << mID << LL_ENDL; -        return false; -    } -    if (index > 0 && index < mTotalPackets-1 && size != MAX_IMG_PACKET_SIZE) -    { -//      LL_WARNS(LOG_TXT) << "Received bad sized packet: " << index << ", " << size << " != " << MAX_IMG_PACKET_SIZE << " for image: " << mID << LL_ENDL; -        return false; -    } - -    if (index >= (S32)mPackets.size()) -    { -        mPackets.resize(index+1, (PacketData*)NULL); // initializes v to NULL pointers -    } -    else if (mPackets[index] != NULL) -    { -//      LL_WARNS(LOG_TXT) << "Received duplicate packet: " << index << " for image: " << mID << LL_ENDL; -        return false; -    } - -    mPackets[index] = new PacketData(data, size); -    while (mLastPacket+1 < (S32)mPackets.size() && mPackets[mLastPacket+1] != NULL) -    { -        ++mLastPacket; -    } -    return true; -} -  void LLTextureFetchWorker::setState(e_state new_state)  {      LL_PROFILE_ZONE_SCOPED_CATEGORY_TEXTURE; diff --git a/indra/newview/lltexturefetch.h b/indra/newview/lltexturefetch.h index aebd2f8f95..3405f76e37 100644 --- a/indra/newview/lltexturefetch.h +++ b/indra/newview/lltexturefetch.h @@ -293,8 +293,6 @@ public:      LLUUID mDebugID;      S32 mDebugCount;      bool mDebugPause; -    S32 mPacketCount; -    S32 mBadPacketCount;      static LLTrace::CountStatHandle<F64>        sCacheHit;      static LLTrace::CountStatHandle<F64>        sCacheAttempt; diff --git a/indra/newview/lltextureview.cpp b/indra/newview/lltextureview.cpp index 92527fc3a9..e5a61b359d 100644 --- a/indra/newview/lltextureview.cpp +++ b/indra/newview/lltextureview.cpp @@ -323,13 +323,6 @@ void LLTextureBar::draw()      {          LLGLSUIDefault gls_ui; -        // draw the packet data -//      { -//          std::string num_str = llformat("%3d/%3d", mImagep->mLastPacket+1, mImagep->mPackets); -//          LLFontGL::getFontMonospace()->renderUTF8(num_str, 0, bar_left + 100, getRect().getHeight(), color, -//                                           LLFontGL::LEFT, LLFontGL::TOP); -//      } -          // draw the image size at the end          {              std::string num_str = llformat("%3dx%3d (%2d) %7d", mImagep->getWidth(), mImagep->getHeight(), @@ -620,10 +613,9 @@ void LLGLTexMemBar::draw()      //---------------------------------------------------------------------------- -    text = llformat("Textures: %d Fetch: %d(%d) Pkts:%d(%d) Cache R/W: %d/%d LFS:%d RAW:%d HTP:%d DEC:%d CRE:%d ", +    text = llformat("Textures: %d Fetch: %d(%d) Cache R/W: %d/%d LFS:%d RAW:%d HTP:%d DEC:%d CRE:%d ",                      gTextureList.getNumImages(),                      LLAppViewer::getTextureFetch()->getNumRequests(), LLAppViewer::getTextureFetch()->getNumDeletes(), -                    LLAppViewer::getTextureFetch()->mPacketCount, LLAppViewer::getTextureFetch()->mBadPacketCount,                      LLAppViewer::getTextureCache()->getNumReads(), LLAppViewer::getTextureCache()->getNumWrites(),                      LLLFSThread::sLocal->getPending(),                      LLImageRaw::sRawImageCount,  | 
