[webkit-dev] Smart Pointers Usage Guidelines

Ryosuke Niwa rniwa at apple.com
Mon Aug 21 18:21:27 PDT 2023


> On Aug 21, 2023, at 4:51 PM, Ryosuke Niwa via webkit-dev <webkit-dev at lists.webkit.org> wrote:
> 
>> On Aug 21, 2023, at 4:50 PM, Tim Horton <thorton at apple.com> wrote:
>> 
>>> On Aug 21, 2023, at 4:42 PM, Ryosuke Niwa <rniwa at apple.com> wrote:
>>> 
>>>> On Aug 21, 2023, at 4:41 PM, Darin Adler <darin at apple.com> wrote:
>>>> 
>>>>> On Aug 21, 2023, at 4:39 PM, Ryosuke Niwa <rniwa at apple.com> wrote:
>>>>> 
>>>>> Alternatively, we could add a new member function which returns CheckedPtr like `pageChecked()`.
>>>> 
>>>> Yes, I think that would be a good approach that would complement the static checker.
>>> 
>>> Okay, let’s go with this solution since others have expressed concerns for ref churns in the past.
>> 
>> So, to be clear, this example:
>> 
>>> page()->document()->foo()
>> 
>> 
>> would become:
>> 
>>> page()->documentChecked()->foo()?
>> 
>> (only `checked` for the deepest getter before the complex call)?
> 
> Yes.

Now that I’m thinking about this more, we should probably call this checkedDocument() to be aligned with protectedX pattern.

- R. Niwa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20230821/eae373f0/attachment.htm>


More information about the webkit-dev mailing list