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

Jump to bottom

COMMON: String performance improvements #5335

Merged

sev- merged 2 commits into scummvm:master from elasota:case-change-perf

Sep 20, 2023

Contributor

elasota commented Sep 18, 2023 •

edited

Some performance improvements to avoid unnecessary string construction and copies:

Changes toLowercase and toUppercase to avoid calling makeUnique if the string doesn't require any modification.
Changes FSNode::getName to avoid calling punycode_decodefilename and the subsequent String -> U32String -> String conversions if the filename isn't punycode. This should significantly speed up AdvancedMetaEngineDetection::composeFileHashMap

elasota added 2 commits

September 18, 2023 11:28


          COMMON: Performance improvement for toUppercase and toLowercase: Avoi…

807148c

…d calling makeUnique on strings that aren't modified.


          COMMON: Avoid calling punycode_decodefilename on strings that aren't …

2892a3d

…punycode.

Member

sev- commented Sep 20, 2023

Thanks! Clever approach with the toLowercase/toUppercase.

sev- merged commit 2f19102 into scummvm:master

7 of 8 checks passed

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