-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
DolphinWX: decompress discs to calculate MD5 hash #3047
Conversation
The idea and implementation are good in general, but I think the blob implementations for CISO and WBFS unfortunately make GetDataSize() return the file size instead of the data size. That should be fixed before this is merged. |
b79e0c6
to
559bc59
Compare
@@ -19,7 +19,7 @@ class WbfsFileReader : public IBlobReader | |||
public: | |||
static WbfsFileReader* Create(const std::string& filename); | |||
|
|||
u64 GetDataSize() const override { return m_size; } | |||
u64 GetDataSize() const override; |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
559bc59
to
d6bd57a
Compare
LGTM. |
@@ -44,7 +44,7 @@ CISOFileReader* CISOFileReader::Create(const std::string& filename) | |||
|
|||
u64 CISOFileReader::GetDataSize() const | |||
{ | |||
return GetRawSize(); | |||
return CISO_MAP_SIZE * m_block_size; |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
btw, this needs a rebase. |
Instead of rounding down, round up.
This is a best-effort approach, since the CISO format does not save the original file size.
d6bd57a
to
00aefa5
Compare
LGTM |
DolphinWX: decompress discs to calculate MD5 hash
The main use case of calculating the MD5 hash is to compare it to known-good values.