Skip to content

add CanGc as argument to methods in OscillatorNode and StereoPannerNode#40146

Merged
jdm merged 1 commit intoservo:mainfrom
yerke:replace-remaining-CanGc-note-part-12
Oct 25, 2025
Merged

add CanGc as argument to methods in OscillatorNode and StereoPannerNode#40146
jdm merged 1 commit intoservo:mainfrom
yerke:replace-remaining-CanGc-note-part-12

Conversation

@yerke
Copy link
Copy Markdown
Contributor

@yerke yerke commented Oct 24, 2025

add CanGc as argument to methods in OscillatorNode and StereoPannerNode

Testing: These changes do not require tests because they are a refactor.
Addresses part of #34573.

@yerke yerke requested a review from gterzian as a code owner October 24, 2025 21:05
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 24, 2025
@yerke yerke force-pushed the replace-remaining-CanGc-note-part-12 branch from e141e85 to e0cd636 Compare October 24, 2025 21:10
@yerke
Copy link
Copy Markdown
Contributor Author

yerke commented Oct 24, 2025

@jdm Here is one for OscillatorNode and StereoPannerNode

@yerke yerke force-pushed the replace-remaining-CanGc-note-part-12 branch from e0cd636 to e1138ff Compare October 24, 2025 23:48
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Oct 25, 2025
@jdm jdm enabled auto-merge October 25, 2025 00:07
@jdm jdm added this pull request to the merge queue Oct 25, 2025
@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 Oct 25, 2025
Merged via the queue into servo:main with commit f16e7b8 Oct 25, 2025
30 checks passed
@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 Oct 25, 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.

3 participants