Bug 230508 - Rename {checked,dynamic}_ns_cast<> to {checked,dynamic}_objc_cast<>
Summary: Rename {checked,dynamic}_ns_cast<> to {checked,dynamic}_objc_cast<>
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Template Framework (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords: InRadar
Depends on: 139893 139915 230227
Blocks:
  Show dependency treegraph
 
Reported: 2021-09-20 14:01 PDT by David Kilzer (:ddkilzer)
Modified: 2021-09-21 15:13 PDT (History)
19 users (show)

See Also:


Attachments
Patch v1 (44.56 KB, patch)
2021-09-20 14:29 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff
Patch v1 (showing changes in renamed files) (36.68 KB, patch)
2021-09-20 14:31 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2021-09-20 14:01:28 PDT
Rename {checked,dynamic}_ns_cast<> to {checked,dynamic}_objc_cast<>.

In Bug 230406, Darin writes:
> Did you see my comment about dynamic_objc_cast? I think I prefer that name to dynamic_ns_cast.
Comment 1 Radar WebKit Bug Importer 2021-09-20 14:01:55 PDT
<rdar://problem/83323145>
Comment 2 David Kilzer (:ddkilzer) 2021-09-20 14:29:51 PDT
Created attachment 438729 [details]
Patch v1
Comment 3 David Kilzer (:ddkilzer) 2021-09-20 14:31:58 PDT
Created attachment 438730 [details]
Patch v1 (showing changes in renamed files)

This is the same as "Patch v1" (Attachment #438729 [details]), but only shows changes in renamed files.

Much easier to review, but I'm not sure if EWS will handle it.
Comment 4 David Kilzer (:ddkilzer) 2021-09-20 14:35:45 PDT
(In reply to David Kilzer (:ddkilzer) from comment #3)
> Created attachment 438730 [details]
> Patch v1 (showing changes in renamed files)
> 
> This is the same as "Patch v1" (Attachment #438729 [details]), but only
> shows changes in renamed files.
> 
> Much easier to review, but I'm not sure if EWS will handle it.

Hmmm...I guess svn-apply-patch will handle this just fine.  We should probably change svn-create-patch to use the pure git patch for renamed files to make them easier to review.
Comment 5 David Kilzer (:ddkilzer) 2021-09-21 08:18:44 PDT
Review ping
Comment 6 Brent Fulgham 2021-09-21 14:36:57 PDT
Comment on attachment 438730 [details]
Patch v1 (showing changes in renamed files)

r=me
Comment 7 EWS 2021-09-21 15:13:31 PDT
Committed r282844 (241975@main): <https://commits.webkit.org/241975@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 438730 [details].