svn commit: r328486 - head/usr.bin/fortune/fortune
Pedro Giffuni
pfg at FreeBSD.org
Sun Jan 28 00:48:00 UTC 2018
On 01/27/18 18:08, Ian Lepore wrote:
> On Sat, 2018-01-27 at 22:56 +0000, Conrad Meyer wrote:
>> I don�t think dragonfly has anything to do with this?��If you�re
>> converting
>> bool increments to setting true values in FreeBSD, use
>> �true.���That�s all
>> there is to it.
>>
>> Best,
>> Conrad
> style(9) emphasizes internal consistancy in several places. ļæ½The
> uppercase TRUE/FALSE is currently the style within that code, so it
> makes sense to stick with it. ļæ½If a (somewhat gratuitous) conversion to
> the new style is made, it should be a separate commit just for that,
> emphasizing that it changes just style and not functionality.
Indeed. I didn't want to mix FALSE with false and TRUE with true so I
did the change as is to make the diff smaller and still have everything
look consistent.
Further cleanups (DragonFly already did it) should be done in a
different commit. I personally don't feel it's something important though.
Pedro.
More information about the svn-src-all
mailing list