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

cxx-qt-gen: add Rust generator and writer of extern "C++Qt" #662

Merged
merged 3 commits into from
Aug 25, 2023

Conversation

ahayzen-kdab
Copy link
Collaborator

Related to #577

@ahayzen-kdab ahayzen-kdab force-pushed the 577-generate-rust-signals branch from bb64c79 to dc18add Compare August 11, 2023 17:05
@ahayzen-kdab ahayzen-kdab force-pushed the 577-generate-rust-signals branch 4 times, most recently from 599de7f to 816666a Compare August 18, 2023 14:25
@ahayzen-kdab ahayzen-kdab marked this pull request as ready for review August 18, 2023 14:27
@ahayzen-kdab ahayzen-kdab changed the title WIP: cxx-qt-gen: add Rust generator and writer of extern "C++Qt" cxx-qt-gen: add Rust generator and writer of extern "C++Qt" Aug 18, 2023
@ahayzen-kdab
Copy link
Collaborator Author

No example yet as we need to add support for closures in signal connection as well as function pointers to make them useful. This will likely be similar to how threading is done and #595 can be reused

@ahayzen-kdab ahayzen-kdab force-pushed the 577-generate-rust-signals branch from 816666a to 0515c02 Compare August 24, 2023 08:27
ahayzen-kdab added a commit to ahayzen-kdab/cxx-qt that referenced this pull request Aug 24, 2023
This allows us later to pass through the original method in the
generation.

Related to KDAB#662
@ahayzen-kdab ahayzen-kdab force-pushed the 577-generate-rust-signals branch from 0515c02 to 23e5c4b Compare August 24, 2023 09:29
ahayzen-kdab added a commit to ahayzen-kdab/cxx-qt that referenced this pull request Aug 24, 2023
This allows us later to pass through the original method in the
generation.

Related to KDAB#662
@ahayzen-kdab ahayzen-kdab force-pushed the 577-generate-rust-signals branch from 23e5c4b to d5fe11d Compare August 24, 2023 09:33
ahayzen-kdab added a commit to ahayzen-kdab/cxx-qt that referenced this pull request Aug 24, 2023
This allows us later to pass through the original method in the
generation.

Related to KDAB#662
@ahayzen-kdab ahayzen-kdab force-pushed the 577-generate-rust-signals branch from d5fe11d to 34fa65f Compare August 24, 2023 10:17
ahayzen-kdab added a commit to ahayzen-kdab/cxx-qt that referenced this pull request Aug 24, 2023
This allows us later to pass through the original method in the
generation.

Related to KDAB#662
@ahayzen-kdab ahayzen-kdab force-pushed the 577-generate-rust-signals branch 3 times, most recently from 48e7b61 to 2b6a244 Compare August 24, 2023 16:14
@Be-ing Be-ing force-pushed the 577-generate-rust-signals branch from 2b6a244 to cf5ad21 Compare August 24, 2023 20:43
ahayzen-kdab added a commit to ahayzen-kdab/cxx-qt that referenced this pull request Aug 25, 2023
This allows us later to pass through the original method in the
generation.

Related to KDAB#662
Copy link
Collaborator

@LeonMatthesKDAB LeonMatthesKDAB left a comment

Choose a reason for hiding this comment

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

LGTM

@ahayzen-kdab ahayzen-kdab merged commit 91230cc into KDAB:main Aug 25, 2023
ahayzen-kdab added a commit that referenced this pull request Aug 25, 2023
This allows us later to pass through the original method in the
generation.

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

Successfully merging this pull request may close these issues.

2 participants