🚀 go-pugleaf

Robert G. Hoover wrote:
> 'busy' is unused throughout the rest of the application during execution. 
> When
> the user closes the main window, I set 'terminated' to true and wait until
> '!grabbing'  Sorry for the confusion. I appended the total thread loop 
> below.
> 'busy' is unused. Again, included for future use, not implemented in any 
> other
> part of the program.

Ok.


> 'grab()' calls my camera supplier's video GrabSingleFrame() call which I 
> have
> been assured but cannot validate is thread-safe.

Ok.


> 'process()' does nothing unless check marks are set in additional GUI 
> window. No
> functions are called from this routine as no checkmarks are checked.
> ex. if (VideoForm->BlurImage->Checked) BlurVideoImage();

That does not sound like that function is thread-safe. I'd use a couple 
bools to store the state of the checkboxes, and then check the bools in the 
thread, instead of checking the actual checkboxes.


> 'display()' calls SendDIBitsToDevice()

Where do you get the device handle to pass to SendDIBitsToDevice?


> Really, these threads should run independent of each other (I hope).

Yes they do, however, they access data that is used by both threads, and if 
the access is not written in a thread-safe manner, it will not work 
properly.

Jon