SimpleCanLib icon indicating copy to clipboard operation
SimpleCanLib copied to clipboard

Esp32-s3 version of ThreadSafeQueue.h

Open txf- opened this issue 1 year ago • 1 comments

Hi,

Just a quick question why is the S3 version of this class checking for semaphores before accessing the queues. Aren't Freertos queues already thread safe? IIRC in Freertos, Semaphores themselves are Freertos Queues.

txf- avatar Aug 26 '24 10:08 txf-

Well that's a good question. I had taken the code from somewhere else and it worked well, so I did not really question that part of it (I am not an expert on FreeRTOS). I did some research now and the result seems to confirm your point. With one of the next releases I will try without the semaphores. Thanks, for pointing this out!

Igitigit2 avatar Sep 01 '24 18:09 Igitigit2