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

DolphinQt: limit patch padding to 8 #7785

Merged
merged 1 commit into from Feb 10, 2019

Conversation

2 participants
@Tilka
Copy link
Member

Tilka commented Feb 5, 2019

32 bits in base 16 are 8 digits, padding to 10 digits makes no sense.

@BhaaLseN

This comment has been minimized.

Copy link
Member

BhaaLseN commented Feb 5, 2019

It is 10 characters if you include 0x in front ;)
Not sure if that was the intent though.

@Tilka Tilka merged commit 33e51da into dolphin-emu:master Feb 10, 2019

9 checks passed

default Very basic checks passed, handed off to Buildbot.
Details
lint Build succeeded on builder lint
Details
pr-android Build succeeded on builder pr-android
Details
pr-deb-dbg-x64 Build succeeded on builder pr-deb-dbg-x64
Details
pr-deb-x64 Build succeeded on builder pr-deb-x64
Details
pr-freebsd-x64 Build succeeded on builder pr-freebsd-x64
Details
pr-ubu-x64 Build succeeded on builder pr-ubu-x64
Details
pr-win-dbg-x64 Build succeeded on builder pr-win-dbg-x64
Details
pr-win-x64 Build succeeded on builder pr-win-x64
Details

@Tilka Tilka deleted the Tilka:patch_editor branch Feb 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment