<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-05-24 10:52 GMT-07:00 Richard <span dir="ltr"><<a href="mailto:legalize@xmission.com" target="_blank">legalize@xmission.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Does LLVM/clang have any style convention preferring:<br>
<br>
if (p)<br>
<br>
over<br>
<br>
if (p != nullptr)<br>
<br>
?<br>
<br>
or it's converse:<br>
<br>
if (!p)<br>
<br>
over<br>
<br>
if (p == nullptr)<br>
<br>
?<br></blockquote><div><br>I don't believe there's any documented (or commonly enforced implicit) convention here.<br><br>I tend towards p/!p, personally. (one place where it's worth being more explicit is in function arguments - "func(p)" where func takes a boolean is a bit subtle, "func(p != nullptr)" seems better there, unless the names of the function/p/etc are /really/ obvious)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I didn't find anything in the existing style guide and I've gotten<br>
a request for readability-simplify-boolean-expr in clang-tidy to<br>
make such simplifications.<br>
<br>
PS: Mail filters are blocking my messages, so ignore this, its<br>
only for them.  Lorem ipsum dolor sit amet, consectetur adipiscing<br>
elit. Phasellus consequat ullamcorper mollis. Sed blandit semper tortor<br>
ultricies dictum. Proin hendrerit et quam in sagittis. Maecenas vel<br>
blandit ante, in auctor sem.  Phasellus condimentum leo vel finibus<br>
viverra. Duis fermentum sollicitudin est, ac iaculis lectus auctor vel.<br>
Nam condimentum nulla feugiat, venenatis nibh a, elementum nulla. Nulla<br>
vitae malesuada eros. Nulla cursus maximus ligula non hendrerit.<br>
<br>
Curabitur lobortis nulla vel sapien posuere, id aliquam orci<br>
bibendum. Vestibulum at vulputate risus. Proin in purus commodo,<br>
tempus lectus vitae, faucibus nunc. Aenean congue faucibus elit, sit<br>
amet facilisis nibh ultrices eget. Nam pulvinar leo ac nunc ultricies,<br>
nec tincidunt nulla tincidunt. Etiam placerat felis tellus, ut ultricies<br>
nunc ultricies quis. In placerat sapien nec ultricies mattis. Integer<br>
sed tempor orci, ac imperdiet orci. Integer ac augue et augue convallis<br>
faucibus.<br>
<span class="HOEnZb"><font color="#888888">--<br>
"The Direct3D Graphics Pipeline" free book <<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__tinyurl.com_d3d-2Dpipeline&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=CnzuN65ENJ1H9py9XLiRvC_UQz6u3oG6GUNn7_wosSM&m=iIrprMmkx40VYYrEco7wyPtAtL9nrtm8BJjpcQKSm3Y&s=6dJU1GnVSzBVMfabQP8u7TnBWoVm1mjCPPKdLcEoHpI&e=" target="_blank">http://tinyurl.com/d3d-pipeline</a>><br>
     The Computer Graphics Museum <<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__ComputerGraphicsMuseum.org&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=CnzuN65ENJ1H9py9XLiRvC_UQz6u3oG6GUNn7_wosSM&m=iIrprMmkx40VYYrEco7wyPtAtL9nrtm8BJjpcQKSm3Y&s=JXXwacVuLsIu4MLCkiq_CmInJ6RXD5HAmpH2mSmVys0&e=" target="_blank">http://ComputerGraphicsMuseum.org</a>><br>
         The Terminals Wiki <<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__terminals.classiccmp.org&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=CnzuN65ENJ1H9py9XLiRvC_UQz6u3oG6GUNn7_wosSM&m=iIrprMmkx40VYYrEco7wyPtAtL9nrtm8BJjpcQKSm3Y&s=WKijLha_SnAf00J0__OIqetN6-fH4Va9yENC4zL6RGQ&e=" target="_blank">http://terminals.classiccmp.org</a>><br>
  Legalize Adulthood! (my blog) <<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__LegalizeAdulthood.wordpress.com&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=CnzuN65ENJ1H9py9XLiRvC_UQz6u3oG6GUNn7_wosSM&m=iIrprMmkx40VYYrEco7wyPtAtL9nrtm8BJjpcQKSm3Y&s=EzJqL3oV9O-e7DmMhSR8BHVVqBk6l2S0D3FBTdQJREA&e=" target="_blank">http://LegalizeAdulthood.wordpress.com</a>><br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</font></span></blockquote></div><br></div></div>