git: e3500c602b13 - main - ipmi: fix negative logic in watchdog control flag

Ed Maste emaste at freebsd.org
Wed Aug 18 15:57:02 UTC 2021


On Wed, 18 Aug 2021 at 02:25, Wojciech Macek <wma at freebsd.org> wrote:
>
> The branch main has been updated by wma:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=e3500c602b13f8252eb8bb779849c41d47306cee
>
> commit e3500c602b13f8252eb8bb779849c41d47306cee
> Author:     Wojciech Macek <wma at FreeBSD.org>
> AuthorDate: 2021-08-18 06:21:14 +0000
> Commit:     Wojciech Macek <wma at FreeBSD.org>
> CommitDate: 2021-08-18 06:21:14 +0000
>
>     ipmi: fix negative logic in watchdog control flag

Thanks!

Avoiding negative logic is something that has been passed on as
developer lore but isn't documented in a particularly obvious way.
Yesterday I made a very small change to sysctl.9 to try to make the
note about -ve logic stand out, but in the coming days I will try to
take a more holistic look at style.9 to make guidance on sysctl naming
and values more clear.


More information about the dev-commits-src-main mailing list