Date: Thu, 25 Jan 2018 22:59:18 -0700 From: Warner Losh <imp@bsdimp.com> To: Matt Joras <matt.joras@gmail.com> Cc: Eitan Adler <eadler@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r328430 - head/sbin/devd Message-ID: <CANCZdfpbfMVYMG0=YxVMA6otHgAqiBu9LT=51RHEf_Ey9x8rzg@mail.gmail.com> In-Reply-To: <CADdTf%2BiVmzMSr%2BdVuXzx-A%2BBPNPLWuT=Rd3CbvEsW5SZOTMFWQ@mail.gmail.com> References: <201801260440.w0Q4efhg008105@repo.freebsd.org> <CANCZdfosuC0CTig=6_p3D5g0fDPhCVW3%2B4HAEbD4EQ4%2B3TetpA@mail.gmail.com> <CADdTf%2BiVmzMSr%2BdVuXzx-A%2BBPNPLWuT=Rd3CbvEsW5SZOTMFWQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jan 25, 2018 at 10:02 PM, Matt Joras <matt.joras@gmail.com> wrote: > On Thu, Jan 25, 2018 at 8:52 PM, Warner Losh <imp@bsdimp.com> wrote: > > > > > > On Thu, Jan 25, 2018 at 9:40 PM, Eitan Adler <eadler@freebsd.org> wrote: > >> > >> Author: eadler > >> Date: Fri Jan 26 04:40:41 2018 > >> New Revision: 328430 > >> URL: https://svnweb.freebsd.org/changeset/base/328430 > >> > >> Log: > >> devd: minor nits > >> > >> - mark usage as noreturn > >> - config does not need a virtual destructor > > > > > > Everything needs a virtual destructor... Please back that part of this > > out... > > > > Warner > Needs? If there's not inheritance there's no _need_ to do it, and it's > arguably superfluous. That being said it's also an arguably > superfluous change to remove it. > OK. Needs might not be the proper word. It's a stylistic thing in all the C++ code I write. It keeps me from having to go back later and add it back when it really does need it after subclasses are derived from it. It's an important detail that often gets overlooked. The cost is minimal. And this isn't one of the rare cases where it's harmful to have it. Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfpbfMVYMG0=YxVMA6otHgAqiBu9LT=51RHEf_Ey9x8rzg>