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

Random blank screen at connection #65

Closed
arvanus opened this issue Aug 17, 2020 · 12 comments
Closed

Random blank screen at connection #65

arvanus opened this issue Aug 17, 2020 · 12 comments

Comments

@arvanus
Copy link
Collaborator

arvanus commented Aug 17, 2020

Randomly occurs:
image
Expected behavior:
image

Anyone got this problem too?
I don't know if it's a build problem, some cross-compile wxWidget error or another thing like that
@mariuz had you gotten this too?

@luronumen
Copy link

HI @arvanus

This issue has happened to me several times but I was unable to identify a sequence of steps that was 100% reproducible.
This is an easier issue to be seen on high latency networks.
In other words, this is a valid issue and I hope you can find the root cause.

Best Regards,
Luciano

@arvanus
Copy link
Collaborator Author

arvanus commented Aug 20, 2020

I think it's something related to the thread while connecting, it's an easy stuff to mess with, and hard do find
😢
Unfortunately I have a little experience with threads in c++, and as it's not really and blocking bug, I won't try to fix it right now

@luronumen
Copy link

Hi @arvanus

Did you have the opportunity to compare the current source code with the source code for version 5ece15b?
I am almost 100% sure that in version 5ece15b this issue would not happen.
Comparing the current code with the 5ece15b code can help you identify the issue.

@arvanus
Copy link
Collaborator Author

arvanus commented Aug 20, 2020

0.9.3.1...master
Looking across versions, I couldn't find much changes
I would list major changes: WINVER change from WINVER=0x400 to WINVER=0x500 and wxWidget change...
The change list aren't that long but off-course I could miss something

@luronumen
Copy link

If we don't have so many changes compared to the 5ece15b version, we are probably seeing this issue because the executable is not optimized.

Lets wait for the @mariuz reply in the issue #38 and retest this issue with the optimized version of flamerobin.

@arvanus
Copy link
Collaborator Author

arvanus commented Aug 21, 2020

I doubt that the lack of optimization is the cause. In fact to much optimization at compiler level can lead to some strange bugs like this small example:
https://developercommunity.visualstudio.com/content/problem/161765/c-compiler-bug-optimization-with-demo-code.html
But of course everything must be tested

@arvanus
Copy link
Collaborator Author

arvanus commented Sep 23, 2020

Looks like don't occurs at debug, just in the "Release Dynamic" build

@blumf
Copy link
Contributor

blumf commented Sep 24, 2020

Can we get a clear picture of what's happening?

  • Does the connection still open?
  • What method of opening is being used (e.g. double-click on registered DB)
  • What version of wxWidgets (check About dialog)

Looks to me like some kind of race condition in wx setting up the dialog. So I guessed at a possible solution in pull req #117

@arvanus
Copy link
Collaborator Author

arvanus commented Sep 24, 2020

Can we get a clear picture of what's happening?

* Does the connection still open?

Yes

* What method of opening is being used (e.g. double-click on registered DB)

double-click

* What version of wxWidgets (check About dialog)

3.1.3

Looks to me like some kind of race condition in wx setting up the dialog. So I guessed at a possible solution in pull req #117

This bug just occurs randomly in release mode,when I'm running in debug mode never occurred
I'd guess it's related to some c++ optimization in release....

@blumf
Copy link
Contributor

blumf commented Sep 24, 2020

* What version of wxWidgets (check About dialog)

3.1.3

Have you tried 3.1.4? Seeing as wx3.1 is the development branch, it's likely they've created some bugs along the way. Seen problems like this before in other desktop GUI libs (Qt, IUP), and it's usually a case of forcing the dialog to layout.

TBH I've just been using the wx debug DLLs for code work, I should really build the static libs.

@arvanus
Copy link
Collaborator Author

arvanus commented Sep 24, 2020

I'll try 3.1.4
I'll give a try to stable 3.0.5 too, (don't know if will compile)

@arvanus
Copy link
Collaborator Author

arvanus commented Jan 25, 2021

Apparently with the new cmake build it's working

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

No branches or pull requests

3 participants