Weldon Washburn wrote: > Your comments helped a bunch. I now understand this code. It looks > correct. It would be good if you could put more comments in the patch. > Maybe rephrasing the below would work. One thing to look out for is a > race > condition that allows a thread to miss an intended notify. > > Also the "#ifdef NO_COND_VARS" is very distracting. Morphing notify_flag > to notify_count is a good idea. But I don't see a reason to attempt to > patch up the NO_COND_VARS use of notify_flag/notify_count. How about > tossing all the code inside #ifdef NO_COND_VARS?
Please find next round of improvements on top of HARMONY-1951 and HARMONY-2269 in HARMONY-2352 (also a subtask of HARMONY-1951).
