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

Bug when moving multiple windows from the same process #13

Open
GitMyCode opened this issue Oct 26, 2015 · 5 comments
Open

Bug when moving multiple windows from the same process #13

GitMyCode opened this issue Oct 26, 2015 · 5 comments

Comments

@GitMyCode
Copy link

Hi,
I created a pull request that explain the problem.
#12

But here it is again

The problem is when we try to move multiple window that comes from the same process. There seems to be a concurrent bug in the VDMHelper for this particular case. When it happen, the windows i tried to move, crash and close itself. It may be isolated to chromium base applications has it doesn't work with multiple instance of chrome, electron.js, atom.io or visual studio code, etc

So the only workaround I found is to move them with a delay between them so the VDMHelper can finish whatever it is doing. It is not the perfect solution but for now it work.

@tmyt
Copy link
Collaborator

tmyt commented Oct 28, 2015

Hi, What environment do you use? (OS Build, 32 or 64bit, etc.)

@GitMyCode
Copy link
Author

windows 10 version 10.0.10240 Build 10240
64bit
System Manufacturer Gigabyte Technology Co., Ltd.
System Model Z68XP-UD3
System Type x64-based PC
System SKU
Processor Intel(R) Core(TM) i5-2500K CPU @ 3.30GHz, 3601 Mhz, 4 Core(s), 4 Logical Processor(s)
BIOS Version/Date Award Software International, Inc. F10, 3/20/2012
SMBIOS Version 2.4
Embedded Controller Version 255.255
BIOS Mode Legacy
BaseBoard Manufacturer Gigabyte Technology Co., Ltd.
BaseBoard Model Not Available
BaseBoard Name Base Board
Platform Role Desktop
Secure Boot State Unsupported
PCR7 Configuration Binding Not Possible
Windows Directory C:\WINDOWS
System Directory C:\WINDOWS\system32
Boot Device \Device\HarddiskVolume1
Locale United States
Hardware Abstraction Layer Version = "10.0.10240.16392"
User Name Desk\Martin
Time Zone Eastern Daylight Time
Installed Physical Memory (RAM) 12.0 GB
Total Physical Memory 11.9 GB
Available Physical Memory 7.41 GB
Total Virtual Memory 16.4 GB
Available Virtual Memory 10.2 GB
Page File Space 4.50 GB
Page File C:\pagefile.sys
Hyper-V - VM Monitor Mode Extensions Yes
Hyper-V - Second Level Address Translation Extensions Yes
Hyper-V - Virtualization Enabled in Firmware Yes
Hyper-V - Data Execution Protection Yes

@tmyt
Copy link
Collaborator

tmyt commented Oct 28, 2015

Ok, I'm going to work to solve this problem.

@tmyt
Copy link
Collaborator

tmyt commented Nov 12, 2015

@GitMyCode fixed this issue in tmyt/VDMHelper@07c0c36. Please check behavior with this commit.

@GitMyCode
Copy link
Author

Hey this is great! By looking a your commit i m pretty sure you got the problem fix.
But i honestly cannot build the VDMHelper project to get the dll. I tried, but i just dont know how, i always get some compile error. If you could put the new dll in this project or directly in the VDMHelper it would be easier for me. Or i will try to build it again later

Good job!

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