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

Calling Reset() after compared files have been closed #27

Closed
Yaron10 opened this issue Mar 23, 2016 · 4 comments
Closed

Calling Reset() after compared files have been closed #27

Yaron10 opened this issue Mar 23, 2016 · 4 comments

Comments

@Yaron10
Copy link

Yaron10 commented Mar 23, 2016

Hello Pavel,

I've tried to zoom in a bit (i.e. I've taken a second look at some parts of Compare.cpp) and found that Reset() is called if the user selects to close both compared-matched files.

It seems that some parts of Reset() are irrelevant in this case.
(Actually , this is true even if the files do not match and the user closes one or both).

I assume you've already noticed that. Just in case. :)

Thank you very much.

Best regards.

@pnedev
Copy link
Owner

pnedev commented Mar 25, 2016

Hello Yaron,

I assume you've already noticed that.

I haven't actually, thanks for pointing this out.
I was busy with other parts of the code and haven't reviewed that part yet.

BR

@Yaron10
Copy link
Author

Yaron10 commented Mar 25, 2016

Hello Pavel,

Thanks again for your remarkable work.

BR

@pnedev
Copy link
Owner

pnedev commented Apr 6, 2016

Hello Yaron,

This is not relevant anymore due to the new implementation so I'm closing this issue.

One question:
What do you think about showing "Close compared files" question dialog in case the user cancels the compare process or if one of the files is empty?
This will also make the implementation more straightforward.
Thanks.

BR

@pnedev pnedev closed this as completed Apr 6, 2016
@Yaron10
Copy link
Author

Yaron10 commented Apr 8, 2016

Hello Pavel,

This is not relevant anymore due to the new implementation so I'm closing this issue.

Great. 👍
Many thanks.

BR

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

2 participants