On 13-10-07 09:13 AM, Dimitar Zhekov wrote:
On Sun, 6 Oct 2013 11:02:07 +1100
Lex Trotman <ele...@gmail.com> wrote:

On 6 October 2013 06:06, Dimitar Zhekov <dimitar.zhe...@gmail.com> wrote:

On Sat, 05 Oct 2013 07:40:15 -0700
Matthew Brush <mbr...@codebrainz.ca> wrote:

So with the pull request adding system() as a fallback/hidden
preference, maybe we could just drop the win32 API code altogether,
switch back to using the same codepath as all platforms (ie. GLib async
spawning) and if people experience issues, we can use as further data
points to troubleshoot GLib async spawning and we can recommend they try
the system() option[1] as a workaround?

Scope uses g_spawn_async_with_pipes() on Win~1 to both read from and
write to gdb without any problems that I know of. Could be because of
the specially written event source, or simply my ignorance. Also
can't remember whether I specially tested stderr.


Or possibly works because its just short messages, IIRC one of the problems
only showed up with lots of data.


Unlikely. It's fully async, and even reentrable for the most part, so
the message size should not matter. And since the win~1 was custom, I
tested the *** thing with the order of megabytes while developing it.
BTW, I re-checked stderr, and it works fine.


Any chance you could find time to review and test Geany's implementation to see if anything looks fishy?

OT: Do you have any idea how to make g_spawn_async_with_pipes() to not open a "Command Prompt" window on Windows? My new plugin filters the document text through a subprocess very often and the popup console windows are super annoying :)

Cheers,
Matthew Brush


_______________________________________________
Devel mailing list
Devel@lists.geany.org
https://lists.geany.org/cgi-bin/mailman/listinfo/devel

Reply via email to