cripto-bot opened a new pull request, #1764: URL: https://github.com/apache/commons-lang/pull/1764
## Summary Extracts the surrogate pair restoration logic from `StrBuilder.reverse()` into a dedicated `restoreSurrogatePairs()` method. ## Motivation The surrogate fixup loop was nested inside the main reversal method, increasing structural depth. Extracting it: - Reduces method complexity - Improves readability (surrogate logic is clearly separated) - Enables independent testing of the surrogate restoration ## Change - Extracted 11 lines of surrogate fixup into `private void restoreSurrogatePairs(char[], int)` - Called from `reverse()` when surrogates are detected - **No behavioral change** — identical logic, same execution path ## Testing All existing `StrBuilderTest` tests should pass. The change is purely structural — same operations in same order. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
