Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Object/Widget inspector Search Bar doesn't Select anything in Large Applications #842

Open
psacawa opened this issue Oct 3, 2023 · 3 comments

Comments

@psacawa
Copy link

psacawa commented Oct 3, 2023

Thanks for your work on this invaluable inspection tool.

When inspecting widgets on large applications, e.g. wireshark, the search bar in the object inspector and widget inspector do not work. It you enter a filter there, the QTreeView will just show all the widgets in the whole application, irrespective of the query.

In smaller applications, this functionality works fine. But it's in large applications where it's actually necessary. I guess the issue is an operation timed out in the search bar. I can see in the source ui/searchlinecontroller.cpp there are QTimers used, but I can't investigate further because gammaray can't inspect itself.

I observed the regression with:

GammaRay version 2.99.50 (revision: 8f2dfd4eb)
Copyright (C) 2010-2023 Klaralvdalens Datakonsult AB, a KDAB Group company, info@kdab.com
Protocol version 38
Broadcast version 2

Using probe qt5_15-x86_64 (Qt 5.15 (x86_64)) on Ubuntu 22.10 to probe wireshark.

I did not observe the regression with:

GammaRay version 2.11.3
Copyright (C) 2010-2021 Klaralvdalens Datakonsult AB, a KDAB Group company, info@kdab.com
Protocol version 36
Broadcast version 2

using the same probe on the same client.

@psacawa
Copy link
Author

psacawa commented Oct 11, 2023

I could also get rid of the issue by rebuild wireshark with Qt6.3, so the issue may be only with Qt5

@Waqar144
Copy link
Contributor

In the latest version (3.0) the filter tries to search + expand the tree, perhaps something is broken there.

@mattkdab
Copy link
Member

@Waqar144 I've been looking into this today. I think the issue may have something to do with the delayedExpandTimer, as I was able to change the behavior but not fully resolve by lengthening the time interval. However, building both GammaRay and Wireshark against Qt 6.6 without making changes to GR sources also resolves this, which is kind of weird. I'm going to take a closer look at the onSearchFinished and tree expansion algorithms this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants