Bug 258418
| Summary: | Simplify a case statement in Decimal.cpp slightly | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | karlcow, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=127667 | ||
Ahmad Saleem
Hi Team,
This is more of clean-up based on Blink's commit.
Commit: https://src.chromium.org/viewvc/blink?view=revision&revision=164071
Just wanted to merge it in WebKit, since it is just clean-up.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Karl Dubost
The description in Chromium
> Simplify a case statement in Decimal.cpp slightly.
>
> Handling for StateDot and StateDotDigit was identical save for a
> "state = StateDotDigit;" in the StateDot case that's a no-op in the
> StateDotDigit case anyway.
https://github.com/WebKit/WebKit/blob/5fa64072e2f447b7ec80714e07bd5c4380c2714f/Source/WebCore/platform/Decimal.cpp#L752-L778
EWS
Committed 265567@main (2b9d1a6ea04f): <https://commits.webkit.org/265567@main>
Reviewed commits have been landed. Closing PR #15211 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/111419452>