-
Notifications
You must be signed in to change notification settings - Fork 174
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
Fix deprecated usage of multiple keys in QHash and QMap. #1468
Conversation
The following links are available: build (macOS-10.15, full) build (ubuntu-18.04, full)
build (windows-latest, full) build (self-hosted_debian-11_aarch64, full)
|
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.
hm these white space changes might cause some annoying conflicts with other PRs but otherwise ok
I also wonder if these Maps were actually originally supposed to be Maps or if they were intended to be Hashes
@HifiExperiments Yeah, the space stuff isn't ideal but that's easy enough to fix if anything conflicts. Also makes things a bit less annoying, as it avoids the need to mess with the whitespace by hand. I think maybe we could just do a giant whitespace tidying commit some day and then install a rule that from now on, it must be perfect. |
…d QMap This replaces those uses with QMultiHash and QMultiMap
33ac65c
to
d77c8bb
Compare
uniqueKeys() is deprecated, and _resources shouldn't contain duplicated keys.
The following links are available: build (macOS-10.15, client) build (ubuntu-18.04, full)
build (windows-latest, full) build (self-hosted_debian-11_aarch64, full)
|
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.
The Interface on Windows and macOS runs, the server runs on Windows.
Fix MultiHash issues added in PR #1468.
This replaces those uses with QMultiHash and QMultiMap. This removes a fair amount of warnings. Only 157 to go.