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

crash when trying to change edge permissions #15776

Open
Tracked by #15574
namdre opened this issue Nov 25, 2024 · 1 comment
Open
Tracked by #15574

crash when trying to change edge permissions #15776

namdre opened this issue Nov 25, 2024 · 1 comment
Assignees
Labels
a:netedit bug dev_regression Works previously in this version p:important more important than default but not 'critical'
Milestone

Comments

@namdre
Copy link
Contributor

namdre commented Nov 25, 2024

Appears after #15725

@namdre namdre added a:netedit bug dev_regression Works previously in this version p:important more important than default but not 'critical' labels Nov 25, 2024
@palvarezlopez palvarezlopez added this to the 1.22.0 milestone Nov 25, 2024
@palvarezlopez
Copy link
Member

palvarezlopez commented Nov 25, 2024

The current implementation has a race condition between set attribute and destroy every row (due a historical reasons). GNEFrameAttributeModules::AttributesEditor has to be updated to maintain always a maximum number of rows (for example, 64) instead of destroy it in every inspection.

palvarezlopez added a commit that referenced this issue Nov 27, 2024
palvarezlopez added a commit that referenced this issue Nov 28, 2024
palvarezlopez added a commit that referenced this issue Nov 28, 2024
…and GNEFrameAttributeModules::AttributesEditorRow. Refs #15776
palvarezlopez added a commit that referenced this issue Nov 28, 2024
palvarezlopez added a commit that referenced this issue Nov 29, 2024
palvarezlopez added a commit that referenced this issue Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:netedit bug dev_regression Works previously in this version p:important more important than default but not 'critical'
Projects
None yet
Development

No branches or pull requests

2 participants