-
Notifications
You must be signed in to change notification settings - Fork 93
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
Fix Search Source File
crashing when used with a complex filter
#2213
Conversation
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
I will review over the next day or two. Is it worth creating or updating any test cases for this fix? |
Search Source File
crahsing when used with a complex filterSearch Source File
crashing when used with a complex filter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments. I personally need more context to understand the purpose of this additional property (sorry!)
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
Here, have a few tests 😊 |
Changes
Fixes #2207
Searching for text in members of a source file would crash if run on a "complex" filter. By complex, I mean a filter whose member filtering term is either a Regex or not a single generic name (
XXX*
).This PR fixes this issue, making sure the search runs fine in all cases.
It also fixes some minor issues:
Listing members...
even after the listing was done andgrep
was runningHow to test this PR
In both cases, the results must be as expected and only the members actually listed by the filter must be shown in the search result.
Checklist