[kate] [Bug 473425] LSP doesn't detect methods used in SIGNAL()/SLOT() macros when searching for references

2023-08-18 Thread Christoph Cullmann
https://bugs.kde.org/show_bug.cgi?id=473425

--- Comment #5 from Christoph Cullmann  ---
I think in the patch below one sees that the handling is in there libclang
based backend

https://codereview.qt-project.org/c/qt-creator/qt-creator/+/227189

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 473425] LSP doesn't detect methods used in SIGNAL()/SLOT() macros when searching for references

2023-08-18 Thread Ahmad Samir
https://bugs.kde.org/show_bug.cgi?id=473425

--- Comment #4 from Ahmad Samir  ---
(In reply to Waqar Ahmed from comment #3)
> clangd provides AST in an xml like format which you can use to get more info
> about the current file and maybe do some extra operations on it. Maybe
> that's how qtcreator is able to achieve this. But afair, qtcreator has a
> third cplusplus backend based on a small c++ parser so maybe it's using that.
> 
> For the issue itself, I generally use search plugin and it works nicely for
> finding refs even in SIGNAL()/SLOT() macros or
> QMetaObject::invokeMethod("stringMethod") stuff.

Yeah, same here I use the search plugin; but sometimes while refactoring code I
might forget, or the code supports multiple platforms with the help of #ifdefS,
so I want to only see the parts relevant to Linux ...etc.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 473425] LSP doesn't detect methods used in SIGNAL()/SLOT() macros when searching for references

2023-08-18 Thread Waqar Ahmed
https://bugs.kde.org/show_bug.cgi?id=473425

Waqar Ahmed  changed:

   What|Removed |Added

 CC||waqar@gmail.com

--- Comment #3 from Waqar Ahmed  ---
clangd provides AST in an xml like format which you can use to get more info
about the current file and maybe do some extra operations on it. Maybe that's
how qtcreator is able to achieve this. But afair, qtcreator has a third
cplusplus backend based on a small c++ parser so maybe it's using that.

For the issue itself, I generally use search plugin and it works nicely for
finding refs even in SIGNAL()/SLOT() macros or
QMetaObject::invokeMethod("stringMethod") stuff.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 473425] LSP doesn't detect methods used in SIGNAL()/SLOT() macros when searching for references

2023-08-18 Thread Ahmad Samir
https://bugs.kde.org/show_bug.cgi?id=473425

--- Comment #2 from Ahmad Samir  ---
I see; I thought since Qt Creator has switched to clangd (instead of libclang
backend) that this works with clangd too; perhaps they have another way of
listing references to methods wrapped in SIGNAL/SLOT.

Sorry for the noise.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 473425] LSP doesn't detect methods used in SIGNAL()/SLOT() macros when searching for references

2023-08-17 Thread Christoph Cullmann
https://bugs.kde.org/show_bug.cgi?id=473425

Christoph Cullmann  changed:

   What|Removed |Added

 Status|REPORTED|RESOLVED
 CC||cullm...@kde.org
 Resolution|--- |NOT A BUG

--- Comment #1 from Christoph Cullmann  ---
As LSP works via clangd, I don't see that we can do that, if one wants that,
this would need to be handled in clangd.
Kate doesn't have a direct clang backend, and I think it should stay that way
given the amount of manpower that would need.

-- 
You are receiving this mail because:
You are watching all bug changes.