Skip to content

script_bindings: Use PhantomData in UnrootedDom and implement Debug and Clone - #46499

Merged
mrobinson merged 1 commit into
servo:mainfrom
mrobinson:phantom-data
Jul 14, 2026
Merged

script_bindings: Use PhantomData in UnrootedDom and implement Debug and Clone#46499
mrobinson merged 1 commit into
servo:mainfrom
mrobinson:phantom-data

Conversation

@mrobinson

@mrobinson mrobinson commented Jul 14, 2026

Copy link
Copy Markdown
Member

This decreases the size of UnrootedDom from 16 to 8 bytes. In
addition, Debug and Cloneare implemented for UnrootedDom
and rustdoc is updated and cleaned up a bit.

Testing: This should not change behavior, so is covered by existing tests.

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jul 14, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jul 14, 2026
…bug` and `Clone`

This decreases the size of `UnrootedDom` from 16 to 8 bytes. In
addition, `Debug` is implemented for `UnrootedDom` and rustdoc is updated
and cleaned up a bit.

Signed-off-by: Martin Robinson <mrobinson@igalia.com>
@mrobinson
mrobinson requested a review from gterzian as a code owner July 14, 2026 15:22
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jul 14, 2026
@mrobinson
mrobinson enabled auto-merge July 14, 2026 15:27
@mrobinson
mrobinson added this pull request to the merge queue Jul 14, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 14, 2026
Merged via the queue into servo:main with commit 86c645b Jul 14, 2026
35 checks passed
@mrobinson
mrobinson deleted the phantom-data branch July 14, 2026 17:48
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants