grag38 / qextserialport

Automatically exported from code.google.com/p/qextserialport
0 stars 0 forks source link

Thread Mutex #18

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Hi everybody,

Well i have been using the old qextserialpor that is in the SF site...

I just want to ask something: Why are you using a mutex in some function of
the Win_QextSerialPort class and in the  Win_QextSerialThread class that is
the thread that can access to variables that are critical it has no mutex?.
I see that the mutex for the _BytesToWrite variable is added but for the
rest variables no, e.g. Win_Handle.

One more thing, In the version that i am using(i know is old, but i have
made some changes), when i call the delete qextserialport object usually a
deadlock appear.

By the way Thanks for such as nice lib

Original issue reported on code.google.com by cafu2...@gmail.com on 21 Sep 2009 at 7:55

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
Is this no important?????

Original comment by cafu2...@gmail.com on 29 Sep 2009 at 9:03

GoogleCodeExporter commented 8 years ago
There is not comments about it, I think is important question, what people 
think??

Original comment by cafu2...@gmail.com on 12 Oct 2009 at 7:07

GoogleCodeExporter commented 8 years ago
Please submit a patch if you had some ideas for improvements - thanks!

Original comment by lst...@gmail.com on 24 Oct 2009 at 11:46

GoogleCodeExporter commented 8 years ago
hi Istack,

i have not made the implematation, cause i dont know why is this done in this 
way, i
would like to know if is intend and why. or if perhaps implementing will give 
any
pros or cons

Original comment by cafu2...@gmail.com on 29 Oct 2009 at 4:20

GoogleCodeExporter commented 8 years ago
Sorry Istask

Original comment by cafu2...@gmail.com on 30 Oct 2009 at 3:28

GoogleCodeExporter commented 8 years ago
This is now protected with the same mutex.

Original comment by lst...@gmail.com on 16 Nov 2009 at 10:30