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.

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

Consistent crashing creating new file after using Pegbar Alignment under specific conditions #1428

Closed
Jose-Moreno opened this issue Aug 17, 2020 · 1 comment · Fixed by #1431
Assignees
Milestone

Comments

@Jose-Moreno
Copy link
Member

Jose-Moreno commented Aug 17, 2020

Issue Summary

Pencil2D crashes once you use the pegbar alignment feature within a file that has at least 2 new layers of the same type (bitmap). For this crash to be reproducible you must create a new bitmap layer by hand or use another feature that automatically creates a new layer (such as the Image Predefined Set, see extra video below)

I believe this could also be tangentially related to the layer naming mismatch problem as reported here: #1414

Expected Results

Pencil2D should not crash when using the pegbar alignment feature nor after creating a new file for any reason.

Video or Image Reference

Minimum reproduction steps.
https://youtu.be/PjbLtdR-C2I

Pencil2D Crashing after importing with Image Predefined Set and using the Pegbar alignment feature.
https://youtu.be/V7ctw3lN0tQ

Steps to reproduce

  1. Create new file
  2. Create a new bitmap layer
  3. Go to Edit > Pegbar Alignment > a dialog will open
  4. Close the dialog immediately
  5. Go to File > New
  6. Silent crash

System Information

  • Pencil2D Version:
    Version: 0.6.5

commit: 50a0c7c
date: 2020-08-01_01:32:15

Operating System: Windows 10 (10.0)
CPU Architecture: x86_64
Qt Version: 5.12.9

@davidlamhauge
Copy link
Contributor

davidlamhauge commented Aug 17, 2020

I looked into this yesterday, and there is a fairly simple solution, so I've assigned my self.
The problem is to close the dialog properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants