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

A bug about layer undo #1860

Closed
puslim opened this issue Sep 18, 2024 · 2 comments · Fixed by #1861
Closed

A bug about layer undo #1860

puslim opened this issue Sep 18, 2024 · 2 comments · Fixed by #1861
Assignees
Labels
Milestone

Comments

@puslim
Copy link

puslim commented Sep 18, 2024

  1. create a layer
  2. create a box on the layer
  3. delete the layer
  4. press undo
  5. create new layer
    After those steps , I found the box is changing into the new layer
    I thought there is something wrong of dbLayoutLayers.cc:263(LayoutLayers::do_insert_layer).
    Maybe m_free_indices should find the index and remove it.
@klayoutmatthias
Copy link
Collaborator

You're right. Perfectly reproducible.

I will debug the issue. Thanks for reporting it.

Matthias

@klayoutmatthias
Copy link
Collaborator

You were right with your analysis. Good catch. The bug must be have been there forever. I wonder how come no one noticed ...

Thanks,

Matthias

klayoutmatthias added a commit that referenced this issue Sep 20, 2024
Fixed issue #1860 (problem with undo after delete layer)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants