On Tue, Mar 05, 2024 at 09:39:11AM +0530, shveta malik wrote: > ConditionVariableTimedSleep() accepts a timeout parameter, but it > doesn't explicitly state the unit for the timeout anywhere. To > determine this, one needs to look into the details of the function to > find it out from the comments of the internally called function > WaitLatch(). It would be beneficial to include a comment in the header > of ConditionVariableTimedSleep() specifying that the timeout is in > milliseconds, similar to what we have for other non-static functions > like WaitLatch and WaitEventSetWait. Attached the patch for the same.
That sounds like a good idea to me, so I'm OK with your suggestion. -- Michael
signature.asc
Description: PGP signature
