[all-commits] [llvm/llvm-project] 59e551: [clangd] Fix renaming single argument ObjC methods...

[David Goldman via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 59e5519c81c57a66424d657864ce69cb0efdc7d8
      https://github.com/llvm/llvm-project/commit/59e5519c81c57a66424d657864ce69cb0efdc7d8
  Author: David Goldman <dallasftball at gmail.com>
  Date:   2024-02-23 (Fri, 23 Feb 2024)

  Changed paths:
    M clang-tools-extra/clangd/refactor/Rename.cpp
    M clang-tools-extra/clangd/unittests/RenameTests.cpp

  Log Message:
  -----------
  [clangd] Fix renaming single argument ObjC methods (#82396)

Use the legacy non-ObjC rename logic when dealing with selectors that
have zero or one arguments. In addition, make sure we don't add an extra
`:` during the rename.

Add a few more tests to verify this works (thanks to @ahoppen for the
tests and finding this bug).



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications