-
Notifications
You must be signed in to change notification settings - Fork 22
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
crashing application #46
Comments
There should be a log file in |
hmm yes i can locate the log but i see nothing related there u have a look 2016-09-19 17:23:39 ERROR GoToWindow - Error while trying to check for updates 2016-09-19 20:08:26 INFO GoToWindow - Application started. Shortcut is 'Left Win + Tab' (5B+09:1) |
You are right, there is really nothing there that helpful. Though it does mean that it does not crash in the GoToWindow code. You can try making the logs more verbose by changing from |
You can also try deactivating plugins, and run as admin / don't run as admin. That will reduce the potential culprits. |
i debugged the project and found the the problem comes when debugger is attached to the window in the GetAppIcon. I solved it with a simple method timeout if you feel comfortable with my change you can accept my pull request |
That makes sense, that would mean it did not "crash", it just got stuck :) I'll review, test and deploy a new version this evening if I can. Thanks for the PR! |
u welcome this the best utility on its kind available today it deserves it! |
I'll keep the issue open until the fix is actually deployed so I don't forget it :) And thanks for the compliment! |
FYI I'd try using https://msdn.microsoft.com/en-us/library/windows/desktop/ms644952(v=vs.85).aspx, if we can avoid spammimng new threads using a built-in api, that would be better... I'll look into it tonight, feel free to try it in the meantime if you want! |
sure anything that solves the case is acceptable, I also tried this http://stackoverflow.com/questions/304109/getting-the-icon-associated-with-a-running-application with no luck. However even if SendMessageWithTimeOut works better i believe a timeout is needed to help if any future problems. It is a shame to lose our time for such rare and not major cases. |
hmm yes i can locate the log but i see nothing related there u have a look 2016-09-19 17:23:39 ERROR GoToWindow - Error while trying to check for 2016-09-19 20:08:26 INFO GoToWindow - Application started. Shortcut is On Wed, Oct 26, 2016 at 7:38 PM, Christian Rondeau <notifications@github.com
Apostolis Bekiaris |
The Release 0.5.2 is now available :) |
There are cases I cannot unfortunately define when just after the search box is displayed and b4 the list is shown the app become unresponsive and crashes. I checked the log but there is nothing related the event viewer entry as well does not help
maybe there is a verbose switch i could use to log ??
The text was updated successfully, but these errors were encountered: