Skip to content

Conversation

@Mrcubix
Copy link
Contributor

@Mrcubix Mrcubix commented Nov 24, 2024

Backport timer changes from #2342 using the worst git usage skills known to man.

@Mrcubix
Copy link
Contributor Author

Mrcubix commented Nov 24, 2024

I hate the linter.

@Mrcubix
Copy link
Contributor Author

Mrcubix commented Nov 24, 2024

I'm getting lint error from stuff done in the original PR, send help.

@Mrcubix
Copy link
Contributor Author

Mrcubix commented Nov 24, 2024

It's still complaining about import ordering, i'm gonna loose my mind

@InfinityGhost InfinityGhost added this to the v0.6.5 milestone Nov 24, 2024
@InfinityGhost InfinityGhost added macos Affects the MacOS platform core OpenTabletDriver core library enhancement New feature or request labels Nov 24, 2024
@Mrcubix
Copy link
Contributor Author

Mrcubix commented Nov 24, 2024

Since the linter doesn't specify which import is causing an issue, i'm unable to fix it
(especially since i reversed the static & alias import and the issue still occurs)

It's the exact same as 406225b again

Co-authored-by: Kuuuube <61125188+Kuuuube@users.noreply.github.com>
@Mrcubix
Copy link
Contributor Author

Mrcubix commented Nov 24, 2024

Forgot to specify :
Closes #3389

But of course someone else is gonna need to add it manually and merging won't close it (happened to me a week ago)

@jamesbt365
Copy link
Member

Theres no point linking it, we will need to manually close it anyway because 0.6.x is not the default branch of the repository.

@gonX gonX linked an issue Nov 24, 2024 that may be closed by this pull request
Copy link
Member

@gonX gonX left a comment

Choose a reason for hiding this comment

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

Code changes match original PR (with appropriate changes for 0.6.x codebase), but would like some testing from a MacOS user.

/e: in less convoluted words: LGTM, would like testing though

@AkiSakurai
Copy link
Contributor

Tested on intel mac, filter is working

Copy link
Member

@gonX gonX left a comment

Choose a reason for hiding this comment

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

Approving given AkiSakurai's testing

@X9VoiD X9VoiD merged commit 35b2dbd into OpenTabletDriver:0.6.x Dec 11, 2024
8 checks passed
@Mrcubix Mrcubix deleted the macos-timer branch December 19, 2024 17:16
@gonX gonX added the forward-port-not-needed Feature is already implemented, is not relevant, or is considered elsewhere for Avalonia branch label Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core OpenTabletDriver core library enhancement New feature or request forward-port-not-needed Feature is already implemented, is not relevant, or is considered elsewhere for Avalonia branch macos Affects the MacOS platform performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Backport MacOS native timer

7 participants