Hi, > Also we usually discourage plain style changes or tweaks unless they are > preparations for more substantial change.
Yes, I'm aware of that. It's a fixture of most projects, and generally a good one IMHO. Besides D43006, there will be several other changes to 'thr_attr.c' (and to 'libthr' more generally). > Can you show the place where it happens? Because the canonical style is > reverse, non-bool values must be explicitly tested instead of relying on > the implicit conversion to bool. I was referring to D40626 and D40642, but it may be considered a different situation because, there, an integer, whatever its actual numerical value, is used as a boolean flag as a whole and there is no '&' to extract a particular bit. Please have a look at the small series starting at D43327 (up to D43330) for the modifications you suggested and a few additional ones of mine. Thanks and regards. -- Olivier Certner
signature.asc
Description: This is a digitally signed message part.
