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
Remove some gotos. #1486
Remove some gotos. #1486
Conversation
|
||
if (finalise) | ||
{ | ||
if (info_ptr != nullptr) png_free_data(png_ptr, info_ptr, PNG_FREE_ALL, -1); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
What's wrong with the gotos? I think the code is clearer and better with them. |
@@ -322,7 +322,7 @@ bool Copy(const std::string &srcFilename, const std::string &destFilename) | |||
char buffer[BSIZE]; | |||
|
|||
// Open input file | |||
FILE *input = fopen(srcFilename.c_str(), "rb"); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
@magcius Maybe the VideoCommon commit complicates the code too much, but I think the FileUtil changes are good. |
} | ||
} | ||
|
||
// write output | ||
int wnum = fwrite(buffer, sizeof(char), rnum, output); | ||
int wnum = fwrite(buffer, sizeof(char), rnum, output.GetHandle()); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
@dolphin-emu-bot rebuild |
lgtm, if you remove the VideoCommon commit. @dolphin-emu-bot rebuild |
Update on this PR ?? |
@dolphin-emu-bot rebuild |
No description provided.