Xargon/ES-DE
1
0
Fork 0
mirror of https://github.com/RetroDECK/ES-DE.git synced 2024-11-22 06:05:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed an incorrect cropSize calculation which sometimes triggered an assertion.

Browse source
This commit is contained in:
Leon Styhre 2023-03-08 19:47:30 +01:00
parent 787164b438
commit eb9cc282a8
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
es-core/src/components/ImageComponent.cpp
View file

@ -206,13 +206,13 @@ void ImageComponent::coverFitCrop()
mBottomRightCrop = {1.0f, 1.0f};
if (std::round(mSize.y) > std::round(mTargetSize.y)) {
const float cropSize {1.0f - (mTargetSize.y / mSize.y)};
const float cropSize {1.0f - (mTargetSize.y / std::round(mSize.y))};
cropTop(cropSize / 2.0f);
cropBottom(cropSize / 2.0f);
mSize.y = mSize.y - (mSize.y * cropSize);
}
else {
const float cropSize {1.0f - (mTargetSize.x / mSize.x)};
const float cropSize {1.0f - (mTargetSize.x / std::round(mSize.x))};
cropLeft(cropSize / 2.0f);
cropRight(cropSize / 2.0f);
mSize.x = mSize.x - (mSize.x * cropSize);

4
es-core/src/components/VideoFFmpegComponent.cpp
View file

@ -124,13 +124,13 @@ void VideoFFmpegComponent::resize()
mSize = textureSize * cropFactor;
if (std::round(mSize.y) > std::round(mTargetSize.y)) {
const float cropSize {1.0f - (mTargetSize.y / mSize.y)};
const float cropSize {1.0f - (mTargetSize.y / std::round(mSize.y))};
mTopLeftCrop.y = cropSize / 2.0f;
mBottomRightCrop.y = 1.0f - (cropSize / 2.0f);
mSize.y = mSize.y - (mSize.y * cropSize);
}
else {
const float cropSize {1.0f - (mTargetSize.x / mSize.x)};
const float cropSize {1.0f - (mTargetSize.x / std::round(mSize.x))};
mTopLeftCrop.x = cropSize / 2.0f;
mBottomRightCrop.x = 1.0f - (cropSize / 2.0f);
mSize.x = mSize.x - (mSize.x * cropSize);