Date: Sun, 10 Jan 2021 10:26:55 +0100 From: Hans Petter Selasky <hps@selasky.org> To: Ryan Libby <rlibby@freebsd.org>, Vladimir Kondratyev <wulf@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: afd590d9e566 - main - hid: Import hidmap and bunch of drivers based on it Message-ID: <d781282f-46ad-e55d-c4bc-7cd4a2b88b80@selasky.org> In-Reply-To: <CAHgpiFx0%2B9tOHjPv9yUbzNQdywxxUm3=FkqK3HBEnWJaOCVQxA@mail.gmail.com> References: <202101072320.107NKosW063030@gitrepo.freebsd.org> <CAHgpiFx0%2B9tOHjPv9yUbzNQdywxxUm3=FkqK3HBEnWJaOCVQxA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 1/10/21 7:39 AM, Ryan Libby wrote: > This broke the gcc build in about a half dozen places due to -Wswitch, > e.g. here. If you like, I can prepare an inelegant patch adding > either default cases or all the ignored cases to all of them. wulf: I believe adding "default:" cases to all switch statements added in the patch series is the right thing to do. --HPS
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?d781282f-46ad-e55d-c4bc-7cd4a2b88b80>