Skip to content

Conversation

@mvorisek
Copy link
Contributor

@mvorisek mvorisek commented Nov 14, 2025

extracted from #19802

All changes are whitespaces, they can be verified by diffing replaced file before/after after replacing spaces with nothing (empty string).

The file was introduced in #18413 from https://github.com/SakiTakamachi/xsse/blob/xsse-1.0.0/src/xsse.h.

@mvorisek
Copy link
Contributor Author

Hi @SakiTakamachi, can you review this PR?

@mvorisek
Copy link
Contributor Author

It seems Saki is not active recently, can someone else please verify correctness of this PR (WS changes) and merge it?

@mvorisek
Copy link
Contributor Author

mvorisek commented Dec 7, 2025

@ndossche can you review please?

Copy link
Member

@ndossche ndossche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified by pulling this locally and replacing the file with upstream

@ndossche ndossche merged commit 88a323e into php:master Dec 7, 2025
10 checks passed
@mvorisek mvorisek deleted the fix_xsse_sync_whitespaces branch December 7, 2025 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants