Skip to content

Conversation

@akyrtzi
Copy link
Contributor

@akyrtzi akyrtzi commented Jul 7, 2022

e4ff5df

[makeprop.pl] Make sure that `CSSPropertyNames.gperf` is generated in a deterministic manner
https://bugs.webkit.org/show_bug.cgi?id=242457
<rdar://96607159>

Reviewed by Fujii Hironori.

Previously the switch-case statements for the properties were written out in a non-deterministic order.

* Source/WebCore/css/makeprop.pl:

Canonical link: https://commits.webkit.org/252297@main

@akyrtzi akyrtzi force-pushed the eng/The-CSSPropertyNames-cpp-file-is-code-generated-with-non-deterministic-contents branch from 7fa106f to 2ae69d2 Compare July 8, 2022 20:54
@fujii fujii added the merge-queue Applied to send a pull request to merge-queue label Jul 8, 2022
… a deterministic manner

https://bugs.webkit.org/show_bug.cgi?id=242457
<rdar://96607159>

Reviewed by Fujii Hironori.

Previously the switch-case statements for the properties were written out in a non-deterministic order.

* Source/WebCore/css/makeprop.pl:

Canonical link: https://commits.webkit.org/252297@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/The-CSSPropertyNames-cpp-file-is-code-generated-with-non-deterministic-contents branch from 2ae69d2 to e4ff5df Compare July 8, 2022 22:02
@webkit-early-warning-system webkit-early-warning-system merged commit e4ff5df into WebKit:main Jul 8, 2022
@webkit-commit-queue
Copy link
Collaborator

Committed 252297@main (e4ff5df): https://commits.webkit.org/252297@main

Reviewed commits have been landed. Closing PR #2196 and removing active labels.

@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jul 8, 2022
@akyrtzi akyrtzi deleted the eng/The-CSSPropertyNames-cpp-file-is-code-generated-with-non-deterministic-contents branch July 8, 2022 22:04
dpino pushed a commit to dpino/webkit that referenced this pull request Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants