git_clone/thorvg
1
0
Fork 0
mirror of https://github.com/thorvg/thorvg.git synced 2025-06-13 11:36:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

loader/lottie: correct the frame range.

Browse source 
Fixed some incorrect time-remapping logic,
Also, the last frame should not be taken into account for the drawing.
This commit is contained in:
Hermet Park 2023-09-14 12:05:35 +09:00 • committed by Hermet Park
parent 87dee3c1f7
commit 42d7a5faed
3 changed files with 13 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/loaders/lottie/tvgLottieBuilder.cpp
View file

@ -140,7 +140,7 @@ static void _updateTransform(LottieLayer* layer, int32_t frameNo)
auto transform = layer->transform;
auto parent = layer->parent;
if (parent) _updateTransform(parent, parent->remap(frameNo));
if (parent) _updateTransform(parent, frameNo);
auto& matrix = layer->cache.matrix;
@ -718,7 +718,7 @@ static void _updatePrecomp(LottieLayer* precomp, int32_t frameNo)
{
if (precomp->children.count == 0) return;
frameNo -= precomp->startFrame;
frameNo = precomp->remap(frameNo);
//TODO: skip if the layer is static.
for (auto child = precomp->children.end() - 1; child >= precomp->children.data; --child) {
@ -779,7 +779,7 @@ static void _updateLayer(LottieLayer* root, LottieLayer* layer, int32_t frameNo)
layer->scene = nullptr;
//visibility
if (frameNo < layer->inFrame || frameNo > layer->outFrame) return;
if (frameNo < layer->inFrame || frameNo >= layer->outFrame) return;
_updateTransform(layer, frameNo);
@ -798,20 +798,18 @@ static void _updateLayer(LottieLayer* root, LottieLayer* layer, int32_t frameNo)
layer->scene->transform(layer->cache.matrix);
auto rFrameNo = layer->remap(frameNo);
switch (layer->type) {
case LottieLayer::Precomp: {
_updatePrecomp(layer, rFrameNo);
_updatePrecomp(layer, frameNo);
break;
}
case LottieLayer::Solid: {
_updateSolid(layer, rFrameNo);
_updateSolid(layer, frameNo);
break;
}
default: {
RenderContext ctx;
_updateChildren(layer, rFrameNo, ctx);
_updateChildren(layer, frameNo, ctx);
break;
}
}
@ -826,7 +824,7 @@ static void _updateLayer(LottieLayer* root, LottieLayer* layer, int32_t frameNo)
if (layer->matte.target->scene) layer->scene->composite(cast<Scene>(layer->matte.target->scene), layer->matte.type);
}
_updateMaskings(layer, rFrameNo);
_updateMaskings(layer, frameNo);
//clip the layer viewport
if (layer->refId && layer->w > 0 && layer->h > 0) {
@ -910,7 +908,7 @@ bool LottieBuilder::update(LottieComposition* comp, int32_t frameNo)
{
frameNo += comp->startFrame;
if (frameNo < comp->startFrame) frameNo = comp->startFrame;
if (frameNo > comp->endFrame) frameNo = comp->endFrame;
if (frameNo >= comp->endFrame) frameNo = (comp->endFrame - 1);
//Update root layer
auto root = comp->root;

4
src/loaders/lottie/tvgLottieModel.cpp
View file

@ -145,8 +145,10 @@ void LottieLayer::prepare()
int32_t LottieLayer::remap(int32_t frameNo)
{
if (timeRemap.frames) {
if (timeRemap.frames || timeRemap.value) {
frameNo = comp->frameAtTime(timeRemap(frameNo));
} else {
frameNo -= startFrame;
}
return (int32_t)(frameNo / timeStretch);
}

9
src/loaders/lottie/tvgLottieModel.h
View file

@ -475,7 +475,7 @@ struct LottieComposition
float duration() const
{
return frameDuration() / frameRate; // in second
return frameCnt() / frameRate; // in second
}
int32_t frameAtTime(float timeInSec) const
@ -483,15 +483,10 @@ struct LottieComposition
auto p = timeInSec / duration();
if (p < 0.0f) p = 0.0f;
else if (p > 1.0f) p = 1.0f;
return (int32_t)lroundf(p * frameDuration());
return (int32_t)lroundf(p * frameCnt());
}
uint32_t frameCnt() const
{
return frameDuration() + 1;
}
uint32_t frameDuration() const
{
return endFrame - startFrame;
}