Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Don't interact with the transform layer if the image is a single layer #8149

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Don't interact with the transform layer if the image is a single layer #8149

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
bfe72a2
Don't interact with the transform layer if the image is a single layer.
Districh-ru Dec 17, 2023
dc56af5
Add assertion to get transform layer only for double-layer images, fi…
Districh-ru Dec 24, 2023
c7d6c6e
Fix fheroes2::Resize() for the case it in and out images have differe…
Districh-ru Dec 24, 2023
12fb09f
Merge branch 'ihhub:master' into single-layer-do-not-access-transform
Districh-ru Dec 24, 2023
ca8874d
Add assertions to more places where transform layer is accessed
Districh-ru Dec 24, 2023
c05b8c6
Merge branch 'ihhub:master' into single-layer-do-not-access-transform
Districh-ru Jan 4, 2024
8782450
Apply IWYU suggestion
Districh-ru Jan 4, 2024
8377456
Revert "Apply IWYU suggestion"
Districh-ru Jan 4, 2024
67edb14
Update copyrights ... can it be automated? :)
Districh-ru Jan 4, 2024
03cc7f8
Merge branch 'ihhub:master' into single-layer-do-not-access-transform
Districh-ru Jan 14, 2024
d8fa4ac
Merge branch 'ihhub:master' into single-layer-do-not-access-transform
Districh-ru Jan 20, 2024
9163fef
Merge branch 'single-layer-do-not-access-transform' of https://github…
Districh-ru Jan 20, 2024
07a89ba
Remove redundant code
Districh-ru Jan 20, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 10 additions & 4 deletions src/engine/image.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -503,7 +503,10 @@ namespace fheroes2
if ( !empty() ) {
const size_t totalSize = static_cast<size_t>( _width ) * _height;
memset( image(), value, totalSize );
memset( transform(), static_cast<uint8_t>( 0 ), totalSize );

if ( !_singleLayer ) {
ihhub marked this conversation as resolved.
Show resolved Hide resolved
memset( transform(), static_cast<uint8_t>( 0 ), totalSize );
}
}
}

Expand Down Expand Up @@ -532,8 +535,11 @@ namespace fheroes2
if ( !empty() ) {
const size_t totalSize = static_cast<size_t>( _width ) * _height;
memset( image(), static_cast<uint8_t>( 0 ), totalSize );
// Set the transform layer to skip all data.
memset( transform(), static_cast<uint8_t>( 1 ), totalSize );

if ( !_singleLayer ) {
// Set the transform layer to skip all data.
memset( transform(), static_cast<uint8_t>( 1 ), totalSize );
}
}
}

Expand All @@ -556,7 +562,7 @@ namespace fheroes2

_singleLayer = image._singleLayer;

memcpy( _data.get(), image._data.get(), size * 2 );
memcpy( _data.get(), image._data.get(), _singleLayer ? size : size * 2 );
}

Sprite::Sprite( const int32_t width_, const int32_t height_, const int32_t x_ /* = 0 */, const int32_t y_ /* = 0 */ )
Expand Down