Skip to content

Use Qt macro - #1337

Open
Masa0u0 wants to merge 2 commits into
ros2:rollingfrom
Masa0u0:rolling
Open

Use Qt macro#1337
Masa0u0 wants to merge 2 commits into
ros2:rollingfrom
Masa0u0:rolling

Conversation

@Masa0u0

@Masa0u0 Masa0u0 commented Feb 17, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@Masa0u0 Masa0u0 changed the title Use Qt macro: slots -> Q_SLOTS Use Qt macro Feb 17, 2025

@ahcorde ahcorde left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

HI @Masa0u0,

can you explain what are you fixing here ?

@audrow audrow added the more-information-needed Further information is required label Feb 27, 2025

@sloretz sloretz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use this macro to replace the emit keyword for emitting signals, when you want to use Qt Signals and Slots with a 3rd party signal/slot mechanism.

https://doc.qt.io/qt-6/qobject.html#Q_EMIT

Use this macro to replace the slots keyword in class declarations, when you want to use Qt Signals and Slots with a 3rd party signal/slot mechanism.

https://doc.qt.io/qt-6/qobject.html#Q_SLOTS

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

Labels

more-information-needed Further information is required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants