headers
strainer | 24 Aug 2010 02:47
Picon

Re: Scite (win) output pane: command involkation with %pathext%

Hi Neil,
I just seen this at then of the if(running) test:

line ~1010
else {
		DWORD nRet = ::GetLastError();
		OutputAppendStringSynchronised(">");
		OutputAppendEncodedStringSynchronised
(GetErrorMessage(nRet), codePage);
		WarnUser(warnExecuteKO);
	}

I understand a bit more now your warning about doublefail.
Ill have another look and see if nRet can be vetted at the first fail.

Sorry that last post was a bit long and formatting messed up.

regards,
stariner

--

-- 
You received this message because you are subscribed to the Google Groups "scite-interest" group.
To post to this group, send email to scite-interest <at> googlegroups.com.
To unsubscribe from this group, send email to scite-interest+unsubscribe <at> googlegroups.com.
For more options, visit this group at http://groups.google.com/group/scite-interest?hl=en.
Permalink | Reply | 0 comments |
headers
Neil Hodgson | 24 Aug 2010 03:24
Picon

Re: Re: Scite (win) output pane: command involkation with %pathext%

strainer:

> If the get.prop is a stability issue i understand, but if it is
> just a matter of milliseconds delay from lineinput it seems worth it
> to be able to interactively test the cli cmd, and not have to declare,
> instaniate and update, another field. Which Ive no clue how to do
> anyway :]

   PropSetFile is not thread-safe. While the execution thread is
trying to read a value, the UI thread may be updating the PropSetFile,
causing a deletion or reallocation and then the execution thread will
be trying to read dead, possibly garbaged memory. Maybe it won't fail
while testing but it will sometimes fail probably causing a crash.

   Neil

--

-- 
You received this message because you are subscribed to the Google Groups "scite-interest" group.
To post to this group, send email to scite-interest <at> googlegroups.com.
To unsubscribe from this group, send email to scite-interest+unsubscribe <at> googlegroups.com.
For more options, visit this group at http://groups.google.com/group/scite-interest?hl=en.
Permalink | Reply | 29 comments |
headers
strainer | 24 Aug 2010 19:42
Picon

Re: Scite (win) output pane: command involkation with %pathext%

Thanks for explaining that Neil.
I'll figure out how to make a sstring in executeOnes scope and set it
from NewLineInOutput.
And im thinking of checking first fail's exit code and only performing
the cli try when it means "..cant find file.." to avoid hiding the
relevant exit codes.

--

-- 
You received this message because you are subscribed to the Google Groups "scite-interest" group.
To post to this group, send email to scite-interest <at> googlegroups.com.
To unsubscribe from this group, send email to scite-interest+unsubscribe <at> googlegroups.com.
For more options, visit this group at http://groups.google.com/group/scite-interest?hl=en.
Permalink | Reply | 28 comments |

Gmane