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

Implement cxx_qt::inherit! to allow inheriting methods from super class #363

Merged
merged 14 commits into from
Feb 23, 2023

Conversation

LeonMatthesKDAB
Copy link
Collaborator

@LeonMatthesKDAB LeonMatthesKDAB commented Nov 24, 2022

Initial implementation for #330.
Does not include the friend implementation for the wrapper method yet, so doesn't complete #330.

@LeonMatthesKDAB LeonMatthesKDAB force-pushed the 330-inherit-methods branch 2 times, most recently from 594c00c to 65fbd42 Compare November 29, 2022 19:07
@LeonMatthesKDAB LeonMatthesKDAB force-pushed the 330-inherit-methods branch 2 times, most recently from f0e8d45 to 74c556f Compare December 7, 2022 13:56
@LeonMatthesKDAB LeonMatthesKDAB marked this pull request as ready for review December 8, 2022 14:47
LeonMatthesKDAB and others added 4 commits February 22, 2023 10:31
- Explaining comment in qml_features
- Improved wording in book

Co-authored-by: Be <[email protected]>
- More tests
- Extract rust generation into /generator/rust/inherit.rs
- Avoid wildcard imports
- Clippy lints
- SPDX licenses
Copy link
Collaborator

@ahayzen-kdab ahayzen-kdab left a comment

Choose a reason for hiding this comment

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

LGTM, lets merge this and any further changes can be after!

@LeonMatthesKDAB LeonMatthesKDAB merged commit 0f8d04c into KDAB:main Feb 23, 2023
przempore pushed a commit to przempore/cxx-qt that referenced this pull request Mar 15, 2023
…ss (KDAB#363)

* Parse cxx_qt::inherit! into ParsedInheritedMethod

* cxx_qt::inherit! - add Rust generation

* CombinedIdent: Revert From<Ident> implementation

Use associated function `from_rust_function` instead.

* cxx_qt::inherit: add rust generation

* Allow cxx_qt::inherit on qobjects without base.

Base class defaults to `QObject`.
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.

3 participants