Skip to content

Commit bdae992

Browse files
author
Grok Compression
committed
fix(freebyrd): fix const shared_ptr reset and unused variable warnings
- Remove block.reset() in decodeAndTransform lambda — the shared_ptr is captured by value (const in non-mutable lambda), so reset() is invalid. Early return without reset is sufficient. - Remove unused numRes and tilec variables in postProcess() Fixes CI build failure on Ubuntu with GCC 13.
1 parent 5d57567 commit bdae992

1 file changed

Lines changed: 0 additions & 5 deletions

File tree

src/lib/core/scheduling/freebyrd/SchedulerFreebyrd.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -489,10 +489,7 @@ bool SchedulerFreebyrd::decodeAndTransform(ITileProcessor* tileProcessor)
489489
resFlow->blocks_->nextTask().work(
490490
[this, activePool, block, tccp, cbw, cbh, cacheAll, tileProcessor]() {
491491
if(!success_)
492-
{
493-
block.reset();
494492
return;
495-
}
496493
t1::ICoder* coder = nullptr;
497494
if(block->needsCachedCoder())
498495
{
@@ -598,8 +595,6 @@ bool SchedulerFreebyrd::postProcess(ITileProcessor* tileProcessor)
598595
continue;
599596

600597
auto tccp = tcp->tccps_ + compno;
601-
auto tilec = tileProcessor->getTile()->comps_ + compno;
602-
uint8_t numRes = tilec->nextPacketProgressionState_.numResolutionsRead();
603598

604599
// Freebyrd doesn't fuse DC shift into wavelet, so always apply it
605600
FlowComponent dcComp;

0 commit comments

Comments
 (0)