RESOLVED FIXED 239646
Remove adjustPathForRecentRenamings from VCSUtils.pm, they are not recent anymore
https://bugs.webkit.org/show_bug.cgi?id=239646
Summary Remove adjustPathForRecentRenamings from VCSUtils.pm, they are not recent any...
Kimmo Kinnunen
Reported 2022-04-22 02:55:33 PDT
Remove adjustPathForRecentRenamings from VCSUtils.pm, they are not recent anymore
Attachments
Patch (2.13 KB, patch)
2022-04-22 02:58 PDT, Kimmo Kinnunen
no flags
Patch (2.38 KB, patch)
2022-04-26 00:11 PDT, Kimmo Kinnunen
no flags
Kimmo Kinnunen
Comment 1 2022-04-22 02:58:59 PDT
Kimmo Kinnunen
Comment 2 2022-04-26 00:11:32 PDT
EWS
Comment 3 2022-04-26 22:41:39 PDT
Committed r293498 (250031@main): <https://commits.webkit.org/250031@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 458334 [details].
Radar WebKit Bug Importer
Comment 4 2022-04-26 22:42:12 PDT
Darin Adler
Comment 5 2022-04-27 14:21:03 PDT
Comment on attachment 458334 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=458334&action=review > Tools/Scripts/VCSUtils.pm:-856 > - # The first and second paths can differ in the case of copies > - # and renames. We use the second file path because it is the > - # destination path. Just noticed this comment. Was there a reason to remove it?
Kimmo Kinnunen
Comment 6 2022-04-28 03:58:15 PDT
Comment on attachment 458334 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=458334&action=review >> Tools/Scripts/VCSUtils.pm:-856 >> - # destination path. > > Just noticed this comment. Was there a reason to remove it? It's a stale comment from the era when the logic of parseGitDiffStartLine was inline in this function, and the regex extracted both the first path and second path from the match. There's no "first path" nor "second path" in this context anymore. parseGitDiffStartLine documents that it returns the destination file for each file diff section in a git patch. bug 38628
Note You need to log in before you can comment on or make changes to this bug.