<p dir="ltr">It has been pointed out that this actually occured in 2017. My apologies.</p>
<div class="gmail_extra"><br><div class="gmail_quote">On 17/01/2017 9:50 PM, "Oliver O'Halloran" <<a href="mailto:oohall@gmail.com">oohall@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">"It's possible I missed one, but I did genuinely review all of it"</p>
<p dir="ltr">Cyril Bur, 2016<br>
In a hobart pub, specifically The Winston</p>
<div class="gmail_extra"><br><div class="gmail_quote">On 17/01/2017 8:53 PM, "Michael Ellerman" <<a href="mailto:mpe@ellerman.id.au" target="_blank">mpe@ellerman.id.au</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Cyril Bur <<a href="mailto:cyrilbur@gmail.com" target="_blank">cyrilbur@gmail.com</a>> writes:<br>
<br>
> On Thu, 2017-01-12 at 14:54 +1100, Russell Currey wrote:<br>
>> Symbolic macros are unintuitive and hard to read, whereas octal constants<br>
>> are much easier to interpret.  Replace macros for the basic permission<br>
>> flags (user/group/other read/write/execute) with numeric constants<br>
>> instead, across the whole powerpc tree.<br>
>><br>
>> Introducing a significant number of changes across the tree for no runtime<br>
>> benefit isn't exactly desirable, but so long as these macros are still<br>
>> used in the tree people will keep sending patches that add them.  Not only<br>
>> are they hard to parse at a glance, there are multiple ways of coming to<br>
>> the same value (as you can see with 0444 and 0644 in this patch) which<br>
>> hurts readability.<br>
>><br>
>> Signed-off-by: Russell Currey <ruscur@russell.cc><br>
><br>
> Reviewed-by: Cyril Bur <<a href="mailto:cyrilbur@gmail.com" target="_blank">cyrilbur@gmail.com</a>><br>
<br>
Did you really really review every single change?<br>
<br>
Because if you did then I don't have to, and that would be *great* :)<br>
<br>
cheers<br>
</blockquote></div></div>
</blockquote></div></div>