search

PelionIoT / mbed-trace

mbed trace library
Apache License 2.0
18 stars 15 forks source link

Clarify mutex type requirement #77

Closed tommikas closed 7 years ago

tommikas commented 7 years ago

If trace helpers are used the mutex must be counting. This was previously mentioned in the header but not in the README.

Addresses #74

korjaa commented 7 years ago

can we make it read:

"If helping functions are used the mutex must be recursive (counting) so it can be acquired from a single thread repeatedly."

other than that, awesome!

tommikas commented 7 years ago

Yeah, that's better. I'll update.

korjaa commented 7 years ago

I men't the bold so that you can notice it better but it might actually work well on the actual text too. I mean it's an important note.

tommikas commented 7 years ago

Yeah, I figured but I thought it worth emphasizing anyway. I don't think the bolded text pops up enough to be distracting either, at least as rendered on github.