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

Show context lines flag not always respected #703

Closed
mdell-seradex opened this issue Jul 7, 2022 · 8 comments
Closed

Show context lines flag not always respected #703

mdell-seradex opened this issue Jul 7, 2022 · 8 comments
Labels

Comments

@mdell-seradex
Copy link

I am unsure why this occurs, but sometimes when I do a search, some files show context lines, even though Show context lines is off. Note that I did not toggle the flag, but I did sort the results (depth-first).
image

@doug24
Copy link
Contributor

doug24 commented Jul 7, 2022

I've tried this a number of times and haven't been able to reproduce it. You say you did not toggle the flag, but did you change it after you clicked the Search button? In the current implementation the state of the flag is saved when the search is started, and that value is used until the next search - which is why the tooltip says: "Context line changes will be applied in the next search".

@mdell-seradex
Copy link
Author

mdell-seradex commented Jul 7, 2022

I just discovered that if I select a file that I had not been selected previously, and then expand it, it expands, showing the context lines. If you expand the file first, it does not show the context lines.
I also found that if I double click a previously unselected file, it expands, showing the context lines, and previews the file.
I believe that the former is likely the action that I took, but even if it isn't, perhaps my action causes the code to take the same route.

Aside from that, I was still having an issue with the cancel button not immediately (or at least quickly) stopping the search, however I have now installed the latest version and it seems better. I was running the previous version (v3.0.64.0) and didn't see anything in the release notes for the latest update regarding that.

@doug24
Copy link
Contributor

doug24 commented Jul 7, 2022

OK I do see the problem now. Selecting the file node and showing the preview takes a different path that isn't checking the flag, just the lines before and lines after.

I didn't knowingly make any changes to the Cancel in the latest release. The code checks for cancel many places and usually responds very quickly. If the code is in some long running operation, the operation doesn't get halted, but there should be checks before and after. It's not obvious where that would be, I've fixed all the ones that I have been able to find.

@doug24 doug24 added the bug label Jul 7, 2022
@mdell-seradex
Copy link
Author

Glad to hear you have the flag problem.

I am unsure what to say about the cancel. The one thing that I believe is that when the search takes longer, the cancel takes longer. Basically it seems that the folder list is cached by Windows on the first run, and that cache is eventually cleared if not used. For example, after a reboot, or if I wait a few days between searches.
I have not rebooted since I installed the latest version.
I will log a new issue if I find it still occurs (unless you find something).

It was kind of bad of me to piggy back a second issue on to this.
Sorry about that.

@doug24
Copy link
Contributor

doug24 commented Jul 7, 2022

If you find a situation that is slow or non-responsive to Cancel, any info you can provide would help - all the filter and search parameters, searching archives, types and sizes of files.

@mdell-seradex
Copy link
Author

It doesn't really seem like it is a specific situation, because I did the exact same search with the latest version (after that hypothetical cache was filled), and it seemed to respond as desired. I think with the previous version too, but I no longer have it installed.
Anyway, like I said, I'll log a new ticket if I run into it again, and will include all my settings.
Thank you.

@mdell-seradex
Copy link
Author

Note that I logged #710 regarding the cancel issue.

@doug24
Copy link
Contributor

doug24 commented Aug 20, 2022

Fixed in release v3.0.136

@doug24 doug24 closed this as completed Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants