[Webkit-unassigned] [Bug 210456] dictionaryValueOfType() in WebCoreArgumentCodersMac.mm can be replaced with dynamic_cf_cast<>()

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Apr 13 14:37:40 PDT 2020


https://bugs.webkit.org/show_bug.cgi?id=210456

--- Comment #1 from Darin Adler <darin at apple.com> ---
Those aren’t the same.

The dictionaryValueOfType is OK for use when we don’t know the type.

dynamic_cf_cast is only for use when do know the type.

Please don’t collapse them into one function. Feel free to change callers if they are using the wrong one.

Also, dynamic_cf_cast is named wrong. C++'s dynamic_cast is OK when we don’t know the type, so this is more like checked_static_cf_cast.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20200413/bfcbe45b/attachment.htm>


More information about the webkit-unassigned mailing list