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