Bug 246322 - encode WebCore::Paths using a trailing "end of path" marker instead of an element count prefix
Summary: encode WebCore::Paths using a trailing "end of path" marker instead of an ele...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Cameron McCormack (:heycam)
URL:
Keywords: InRadar
Depends on: 246319
Blocks:
  Show dependency treegraph
 
Reported: 2022-10-10 22:46 PDT by Cameron McCormack (:heycam)
Modified: 2022-10-21 08:57 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Cameron McCormack (:heycam) 2022-10-10 22:46:30 PDT
The decoder doesn't need to know the number of elements ahead of time, and writing a uint8_t marker at the end instead of a uint64_t at the start can save some bytes.
Comment 1 Radar WebKit Bug Importer 2022-10-10 22:47:08 PDT
<rdar://problem/101015632>
Comment 2 Cameron McCormack (:heycam) 2022-10-11 15:57:28 PDT
rdar://100873493
Comment 3 Cameron McCormack (:heycam) 2022-10-11 16:01:03 PDT
Pull request: https://github.com/WebKit/WebKit/pull/5264
Comment 4 EWS 2022-10-12 13:46:17 PDT
Committed 255452@main (09496e855222): <https://commits.webkit.org/255452@main>

Reviewed commits have been landed. Closing PR #5264 and removing active labels.
Comment 5 EWS 2022-10-21 08:57:24 PDT
Committed 252432.632@safari-7614-branch (81b6f2dd592c): <https://commits.webkit.org/252432.632@safari-7614-branch>

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