-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
MusicDataBaseDirectory::NODE_TYPE: convert to enum class #26027
Conversation
5dd18b2
to
7691d0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DirectoryNode.h is included in many header files. Any chance to replace some of the includes with forward declarations to the new enum class now?
Quick check:
- filesystem/MusicdatabaseDirectory/QueryParams.h
- filesystem/VideodatabaseDirectory/QueryParams.h
- filesystem/MusicDatabaseDirectory.h
- filesystem/VideoDatabaseDirectory.h
7691d0f
to
ae59e38
Compare
added another commit with the forwards. the video ones will have to wait until the PR that changes the video enum (next in my queue..) |
and rename NodeType to follow conventions
ae59e38
to
f9f64b6
Compare
f9f64b6
to
65de957
Compare
good to go? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shove it in.
Description
Rename and change an enum to an enum class
Motivation and context
More modern, more type safe, ..
How has this been tested?
It builds
What is the effect on users?
None
Types of change
Checklist: