git: 16079c7233be - main - hid: quiet -Wswitch

[Ryan Libby]

On Mon, Jan 11, 2021 at 10:04 AM John Baldwin <jhb at freebsd.org> wrote:
>
> On 1/10/21 9:54 PM, Ryan Libby wrote:
> > The branch main has been updated by rlibby:
> >
> > URL: https://cgit.FreeBSD.org/src/commit/?id=16079c7233be8bd6c88e3421a70c7ca87cfea370
> >
> > commit 16079c7233be8bd6c88e3421a70c7ca87cfea370
> > Author:     Ryan Libby <rlibby at FreeBSD.org>
> > AuthorDate: 2021-01-11 05:53:15 +0000
> > Commit:     Ryan Libby <rlibby at FreeBSD.org>
> > CommitDate: 2021-01-11 05:53:15 +0000
> >
> >     hid: quiet -Wswitch
> >
> >     Gcc builds complained that not all switch cases are handled.  Add
> >     default cases to appease gcc.
> >
> >     Reviewed by:    hselasky (previous version), wulf
> >     Sponsored by:   Dell EMC Isilon
> >     Differential Revision:  https://reviews.freebsd.org/D28082
>
> If these cases are never reachable, then I think '__assert_unreachable()'
> is preferred to a plain break.
>
> --
> John Baldwin

I'll have to let wulf and hselasky speak to that...