Neil Hodgson wrote:
SetDocumentAt is not really something that should be called from
the tool thread.
This code is within SciTEWin::ExecuteOne(). Maybe I should store the
job output to a variable within the job object and have the
IDM_FINISHEDEXECUTE handler do the replacement.
I think its reasonable to move the replacing to the UI thread as
that means you don't have to call any of the buffer methods. I
wouldn't mind the replacement being ignored, like Bruce says, but even
to ignore it, you want to know the current buffer which is not really
thread-safe.
I've altered the JobQueue class and moved the selection replacement code
into SciTEWin::Command(). The output goes to the proper buffer, and the
output pane does not flicker wildly like the first incarnation did.
I'll upload everything in a few days.
April
--
I am grateful that I am not as judgmental as all those , self-righteous people
around me.
_______________________________________________
Scite-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scite-interest
