Comment on posix condition variable implementation looks bogus

[wucke13]

• COndition -> Condition
• "into a semaphore a cast" ? With the double space between "semaphore" and 'a'?

threadx/utility/rtos_compatibility_layers/posix/px_cond_signal.c

Line 92 in 485a02f

/* Simply convert the COndition variable control block into a semaphore a cast */

[fdesbiens]

Thanks for raising this issue, @wucke13.

Looking at the code, I think the comment should read:

Simply convert the condition variable control block into a semaphore pointer.

What do you think?

[jserv]

The comment can be refined as following:

--- a/utility/rtos_compatibility_layers/posix/px_cond_signal.c
+++ b/utility/rtos_compatibility_layers/posix/px_cond_signal.c
@@ -89,7 +89,7 @@ TX_SEMAPHORE        *semaphore_ptr;
 UINT                 status;
 
     /* Get the condition variable's internal semaphore.  */
-    /* Simply convert the COndition variable control block into a semaphore  a cast */ 
+    /* Simply convert the condition variable control block into a semaphore using casting */
     semaphore_ptr = (&( cond->cond_semaphore ));
     if ( semaphore_ptr->tx_semaphore_suspended_count)
     {

[wucke13]

Seems sensible to me!

[fdesbiens]

@wucke13 Just to confirm I will make the change. I will take care of this next week, as I am on a trip right now.