tom-anders wrote:

> Something else I noticed while trying out the patch locally: before the 
> patch, the description of the code action in the editor is "change 'foo' to 
> 'Foo'", i.e. a description of what the code action will do.
> 
> After the patch, the description of the code action is now "invalid case 
> style for function 'foo'", i.e. it describes the problem, not the fix.
> 
> We should keep the description of the code action the same, i.e. describing 
> the fix.

Ah good find, I added a new `FixMessage` field for this (and also added this to 
the test)

https://github.com/llvm/llvm-project/pull/78454
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to