sammccall added a comment.

Does this regress the case where you complete a file rather than a directory?
e.g. `#include "foo/^one.h"`
If you select "two.h", then we definitely want to replace "one.h", right?

The root here is that we have to decide what to replace independently of the 
completion item.
One compromise might be to replace up to the quote, but if you're completing 
`"foo/^one.h"` and `bar` is a directory, check if `foo/bar/one.h` exists and 
offer it as a completion. Seems complicated though.

Or we can take the tradeoff here if it's better but it'd be good to understand 
why.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76770/new/

https://reviews.llvm.org/D76770



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to