这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@chchwy
Copy link
Member

@chchwy chchwy commented Apr 28, 2022

This is a PR to address the issue mentioned here:
#1690 (review)

Actually ,the app could crash whenever calling BitmapImage::clone() on an unloaded bitmap keyframe.
My solution is to make a copy of the original image file with a temporary filename if the image is unloaded.

@chchwy chchwy force-pushed the fix-crash-duplicate-layer branch from bab1732 to c374b2c Compare April 29, 2022 03:12
Copy link
Member

@MrStevns MrStevns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes looks good to me, although I wasn't able to reproduce the crash that was the reason for the PR, it did however fix a problem where the unloaded frames wouldn't be shown if you set the visibility of the layer to hidden and then duplicated it.

@MrStevns MrStevns merged commit 4ff723b into pencil2d:master May 12, 2022
@chchwy chchwy deleted the fix-crash-duplicate-layer branch May 13, 2022 04:52
@chchwy chchwy added this to the 0.7.0 milestone May 20, 2022
@MrStevns MrStevns modified the milestones: 0.7.0, v0.6.7 Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants