Skip to content

fix: use safe std::string parameters in Qt signals #6

Description

@otto-link

Problem

Some Qt widget signals currently use const std::string& parameters.

This is unsafe with queued Qt connections, as the referenced string may no longer be valid when the signal is delivered.

Proposed solution

Replace all const std::string& signal parameters with std::string passed by value.

Review the corresponding slots, connections, and tests to ensure the change is applied consistently.

Activity

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

Metadata

Metadata

Assignees

Labels

theme: graph-editor-uxNode menu, links, selection, keyboard, layout, clipboard

Type

No type

Fields

Priority

P2 - High

Effort

S

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions