Date: Thu, 22 Feb 2018 07:56:16 +0200 From: Toomas Soome <tsoome@me.com> To: Alexander Motin <mav@FreeBSD.org> Cc: rgrimes@freebsd.org, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-vendor@freebsd.org Subject: Re: svn commit: r329770 - vendor-sys/illumos/dist/uts/common/fs/zfs/lua Message-ID: <14ECCB6A-7AEE-4831-AF14-6AAF79CF4102@me.com> In-Reply-To: <56dda9e2-e9ea-83b9-1e54-6569d478615a@FreeBSD.org> References: <201802220104.w1M14uHQ088614@pdx.rh.CN85.dnsmgr.net> <56dda9e2-e9ea-83b9-1e54-6569d478615a@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On 22 Feb 2018, at 05:03, Alexander Motin <mav@FreeBSD.org> wrote: >=20 > On 21.02.2018 20:04, Rodney W. Grimes wrote: >> [ Charset UTF-8 unsupported, converting... ] >>> Author: mav >>> Date: Thu Feb 22 00:46:24 2018 >>> New Revision: 329770 >>> URL: https://svnweb.freebsd.org/changeset/base/329770 >>>=20 >>> Log: >>> 9035 zfs: this statement may fall through >>>=20 >>> illumos/illumos-gate@46ac8fdfc5a1f9d8240c79a6ae5b2889cbe83553 >>>=20 >>> Reviewed by: Yuri Pankov <yuripv@yuripv.net> >>> Reviewed by: Andy Fiddaman <omnios@citrus-it.co.uk> >>> Reviewed by: Matthew Ahrens <mahrens@delphix.com> >>> Approved by: Dan McDonald <danmcd@joyent.com> >>> Author: Toomas Soome <tsoome@me.com> >>>=20 >>> Modified: >>> vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lgc.c >>> vendor-sys/illumos/dist/uts/common/fs/zfs/lua/llex.c >>> vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lstrlib.c >>> vendor-sys/illumos/dist/uts/common/fs/zfs/lua/ltable.c >>>=20 >>> Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lgc.c >>> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >>> --- vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lgc.c Thu Feb = 22 00:42:12 2018 (r329769) >>> +++ vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lgc.c Thu Feb = 22 00:46:24 2018 (r329770) >>> @@ -677,7 +677,7 @@ static void freeobj (lua_State *L, GCObject *o) = { >>> case LUA_TUSERDATA: luaM_freemem(L, o, sizeudata(gco2u(o))); = break; >>> case LUA_TSHRSTR: >>> G(L)->strt.nuse--; >>> - /* go through */ >>> + /* FALLTHROUGH */ >>> case LUA_TLNGSTR: { >>> luaM_freemem(L, o, sizestring(gco2ts(o))); >>> break; >>>=20 >>> Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/lua/llex.c >>> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >>> --- vendor-sys/illumos/dist/uts/common/fs/zfs/lua/llex.c Thu Feb = 22 00:42:12 2018 (r329769) >>> +++ vendor-sys/illumos/dist/uts/common/fs/zfs/lua/llex.c Thu Feb = 22 00:46:24 2018 (r329770) >>> @@ -475,6 +475,7 @@ static int llex (LexState *ls, SemInfo *seminfo) = { >>> else if (!lisdigit(ls->current)) return '.'; >>> /* else go through */ >> ^^^^^^^^^^^^^^^^^^^^^^^ Wasnt this already marked, perhaps = in the >> wrong place with the wrong string? > As I understand GCC 7 parses comments like that to decide whether > generate warnings or not, but this informal text is out of his > understanding. >=20 >=20 Exactly. Also in some cases it is desirable to have more text than just = FALLTHROUGH, And in some cases we may just have missed the original = comment which should be removed:) rgds, toomas
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?14ECCB6A-7AEE-4831-AF14-6AAF79CF4102>