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
Import all the things! #696
Conversation
launcher/minecraft/mod/DataPack.cpp
Outdated
switch (type) { | ||
default: { | ||
auto res = Resource::compare(other, type); | ||
if (res.first != 0) | ||
return res; | ||
} | ||
case SortType::PACK_FORMAT: { | ||
auto this_ver = packFormat(); | ||
auto other_ver = cast_other.packFormat(); | ||
|
||
if (this_ver > other_ver) | ||
return { 1, type == SortType::PACK_FORMAT }; | ||
if (this_ver < other_ver) | ||
return { -1, type == SortType::PACK_FORMAT }; | ||
} | ||
} |
Check notice
Code scanning / CodeQL
No trivial switch statements
with the latest commit this should fix #700 as well |
This should be rebased |
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
cc3b244
to
47381a1
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.
this looks good to me, just did some nitpicks :3
btw, 47381a1 has some typos in the description :P |
fixed by properly converting from a file path and converting to native seperators. should have known naive handling of file path as a URL would come back to bite us cross platform. Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
47381a1
to
aa550d7
Compare
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
aa550d7
to
574af2c
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.
lgtm, thanks!
Co-authored-by: Sefa Eyeoglu <contact@scrumplex.net> Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
d05365e
to
f7b0ba8
Compare
fixed |
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.
Thanks!
Based off #576
Fixes #581
Fixes #700
-I
flag to be passed multiple times-I
flag and though tacked on at the end without flags are treated the same are functionally identical