W3C home > Mailing lists > Public > public-webrtc-logs@w3.org > March 2022

Re: [mediacapture-region] Make CropTarget serializable (#24)

From: youennf via GitHub <sysbot+gh@w3.org>
Date: Tue, 29 Mar 2022 14:56:18 +0000
To: public-webrtc-logs@w3.org
Message-ID: <issue_comment.created-1081975698-1648565776-sysbot+gh@w3.org>
> As [mentioned](https://github.com/w3c/mediacapture-region/pull/32#issuecomment-1080798232), if you have examples of such "weak references" being employed elsewhere, then we can fall in with this pattern. Otherwise, I think a less controversial phrasing is preferable. Do you have such examples?

Given GC should not be observable, whether weak or not is not a hard requirement, it is a clarification.
If you do not like weak (I am not sure why), I am fine reverting to the original PR: no mention of weak but a GC note.

As of objects referencing other objects that are potentially in other process, this is an existing pattern that proved to work well, let's use it.

GitHub Notification of comment by youennf
Please view or discuss this issue at https://github.com/w3c/mediacapture-region/pull/24#issuecomment-1081975698 using your GitHub account

Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Tuesday, 29 March 2022 14:56:25 UTC

This archive was generated by hypermail 2.4.0 : Saturday, 6 May 2023 21:19:56 UTC