Developer forums (C::B DEVELOPMENT STRICTLY!) > Plugins development
codecompletion plugin lead codeblocks crash, do GUI operation in worker thread
kipade:
Hello ollydbg, I just define a simple event to notify ClassBrowser to call ClassBrowserBuilderThread's BuildTree function, because BuildTree was originally called in a while loop, also has semaphore to guarantee ths order of the operations. So, it works for me, under linux. Although this is not a good idea, it really works. And, this also has not make the GUI busy, almost fell nothing of it.
ollydbg:
--- Quote from: kipade on February 13, 2014, 07:35:15 am ---Hello ollydbg, I just define a simple event to notify ClassBrowser to call ClassBrowserBuilderThread's BuildTree function, because BuildTree was originally called in a while loop, also has semaphore to guarantee ths order of the operations. So, it works for me, under linux. Although this is not a good idea, it really works. And, this also has not make the GUI busy, almost fell nothing of it.
--- End quote ---
Thanks, can you show some patches? So that I can tested on my local PC(winXP).
oBFusCATed:
Probably the best way is to define a custom event where you can store 50 or 100 items. Then in the worker thread you just send these elements and in the main thread you add the items to the tree in small batches. I think this won't block the ui and user events might be process normally.
kipade:
--- Quote from: oBFusCATed on February 13, 2014, 08:47:48 am ---Probably the best way is to define a custom event where you can store 50 or 100 items. Then in the worker thread you just send these elements and in the main thread you add the items to the tree in small batches. I think this won't block the ui and user events might be process normally.
--- End quote ---
Mightbe. But, it need more coding work, right? I just define a custom event and simply move the BuildTree into ClassBrowser window messages processing chain. I also think this operation does need to be processed in this window class, since, all operations of this function is work for ClassBrowser. All of this GUI operations in this function also would not take too much time. If move the responsibility to ClassBrowser, the function should be rewrite, and, some class definition should be redesined,too. I think.
oBFusCATed:
kipade: Have you do some work on this or this is just a suggestion?
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version