================

----------------
argothiel wrote:

Good point, I've added the test cases for Sema, no Sema nad the Lexer.

Also, even though the ranges end with the identifier anyway, I see that there's 
a long-standing issue regarding snippet production, vaguely related to that. 
I'll probably look at it after this PR to make the replace mode more useful:
```cpp
// FIXME(kirillbobyrev): No snippet should be produced here.
EXPECT_THAT(completions(Context + "int y = fo^o(42)", {}, Opts).Completions,
            UnorderedElementsAre(
                AllOf(labeled("foo(int A)"), snippetSuffix("(${1:int A})"))));
```

https://github.com/llvm/llvm-project/pull/187623
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to