Flickering console window when sending Ctrl+C

[segrey]

When a process is started without console window, executing org.jvnet.winp.WinProcess#sendCtrlC leads to running sendctrlc.exe with a new console visible window. Even though sendctrlc.exe is a short-living process, the console window flickering is noticeable.

To reproduce the issue, please see myapp.zip.
It's a simple Java Swing application running new WinProcess(123).sendCtrlC() on button click.

1. Unzip the project and optionally rebuild it.
2. Run the app with run.cmd. Or start it manually, just ensure you're using javaw.exe instead of java.exe to start the process without console window.
3. A window with a single button will appear:
   
4. Click it and a console window will appear and hide.

[segrey]

I have a patch for this issue based on using java.lang.ProcessBuilder and spawn sendctrlc.exe from Java instead of doing it in winp.cpp (SendCtrlC). If no objections, I'll open the PR soon.

[segrey]

The issue affects IDE users: https://youtrack.jetbrains.com/issue/IDEA-202408

[oleg-nenashev]

sure, please feel free to submit it

…

On Wed, Nov 21, 2018 at 11:23 PM Sergey Simonchik ***@***.***> wrote: The issue affects IDE users: https://youtrack.jetbrains.com/issue/IDEA-202408 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC3IoO3PJgr4Du8WAfMFn44AiGbk45ezks5uxdJIgaJpZM4Yt-H6> .

[segrey]

Suggested fix: PR #61

[segrey]

Fixed by be5a29f

[segrey]

Another reported case about flickering window popup: https://youtrack.jetbrains.com/issue/WEB-36921 (IntelliJ IDEA).
@oleg-nenashev Would be great to cut a release.

[oleg-nenashev]

@segrey I am still waiting for the new Windows laptop, sorry :(
Let's proceed with #64 which might be a quicker solution. I might get my laptop next week, but it's hard to guarantee

[segrey]

Sure, next week is fine. Regarding #64, sorry, can it help to cut a release quicker?

[oleg-nenashev]

Let's try to do both. It is apparently not trivial to revert the environment to the WinP-capable version to do the release, and I would not like to spend few days on updating a laptop which should have been decommissioned months ago :( If it was not Maven Central, I would have already automated releases on AppVeyor or other CI system.

…

On Fri, Feb 15, 2019 at 6:24 PM Sergey Simonchik ***@***.***> wrote: Sure, next week is fine. Regarding #64 <#64>, sorry, can it help to cut a release quicker? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC3IoPfHmIexXvIDt-Kz7AnD0nbLBOXmks5vNu1FgaJpZM4Yt-H6> .

[oleg-nenashev]

Fixed in WinP 1.28