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

Officially Deprecate EventBodyQt. #220

Open
TTWNO opened this issue Feb 12, 2025 · 1 comment
Open

Officially Deprecate EventBodyQt. #220

TTWNO opened this issue Feb 12, 2025 · 1 comment

Comments

@TTWNO
Copy link
Member

TTWNO commented Feb 12, 2025

While it may remain part of the public API, invoking its use should elicit a warning about incompatibility with future versions of at-spi2-core and mention the Qt no longer uses this signature. We may also want to change the name to LegacyEventBodyQt, or something similar to indicate its deprecation.

Qt is now officially using the correct signature like GTK does[1], so I'm wanting to make sure no new uses of this structural variant spread throughout our or others' codebases.

  1. https://code.qt.io/cgit/qt/qtbase.git/commit/?id=fdf87fdce1c8b926b1c1ee9254aa4b0fc1840d82
@TTWNO
Copy link
Member Author

TTWNO commented Feb 12, 2025

Where it is required for our own use, we can use explicit ignoring of deprecation using #[allow(...)].

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

No branches or pull requests

1 participant