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

refactor(util/timer): cleanup includes #11937

Merged
merged 3 commits into from
Sep 8, 2023

Conversation

Swiftb0y
Copy link
Member

@Swiftb0y Swiftb0y commented Sep 7, 2023

I noticed an AutoMoc warning during the build and got annoyed by it. Turns out there was an unnecessary #include "moc_timer.cpp". While I was at it, I also cleaned up the rest of the headers.

@Swiftb0y Swiftb0y marked this pull request as draft September 7, 2023 23:20
@Swiftb0y Swiftb0y force-pushed the refactor/cleanup-timer-headers branch from 7b806ae to 87e24b2 Compare September 7, 2023 23:26
@Swiftb0y Swiftb0y force-pushed the refactor/cleanup-timer-headers branch from 87e24b2 to dea19a3 Compare September 7, 2023 23:29
@Swiftb0y Swiftb0y marked this pull request as ready for review September 7, 2023 23:30
@JoergAtGithub
Copy link
Member

Thank you! LGTM!

@JoergAtGithub JoergAtGithub merged commit f4692aa into mixxxdj:main Sep 8, 2023
11 checks passed
@Swiftb0y Swiftb0y deleted the refactor/cleanup-timer-headers branch September 8, 2023 08:35
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