svn commit: r329770 - vendor-sys/illumos/dist/uts/common/fs/zfs/lua

[Rodney W. Grimes]

[ 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
> 
> Log:
>   9035 zfs: this statement may fall through
>   
>   illumos/illumos-gate at 46ac8fdfc5a1f9d8240c79a6ae5b2889cbe83553
>   
>   Reviewed by: Yuri Pankov <yuripv at yuripv.net>
>   Reviewed by: Andy Fiddaman <omnios at citrus-it.co.uk>
>   Reviewed by: Matthew Ahrens <mahrens at delphix.com>
>   Approved by: Dan McDonald <danmcd at joyent.com>
>   Author: Toomas Soome <tsoome at me.com>
> 
> 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
> 
> Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lgc.c
> ==============================================================================
> --- 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;
> 
> Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/lua/llex.c
> ==============================================================================
> --- 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?

>        }
> +      /* FALLTHROUGH */
>        case '0': case '1': case '2': case '3': case '4':
>        case '5': case '6': case '7': case '8': case '9': {
>          read_numeral(ls, seminfo);
> 
> Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lstrlib.c
> ==============================================================================
> --- vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lstrlib.c	Thu Feb 22 00:42:12 2018	(r329769)
> +++ vendor-sys/illumos/dist/uts/common/fs/zfs/lua/lstrlib.c	Thu Feb 22 00:46:24 2018	(r329770)
> @@ -514,7 +514,7 @@ static const char *match (MatchState *ms, const char *
>              }
>              case '+':  /* 1 or more repetitions */
>                s++;  /* 1 match already done */
> -              /* go through */
> +              /* FALLTHROUGH */
>              case '*':  /* 0 or more repetitions */
>                s = max_expand(ms, s, p, ep);
>                break;
> 
> Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/lua/ltable.c
> ==============================================================================
> --- vendor-sys/illumos/dist/uts/common/fs/zfs/lua/ltable.c	Thu Feb 22 00:42:12 2018	(r329769)
> +++ vendor-sys/illumos/dist/uts/common/fs/zfs/lua/ltable.c	Thu Feb 22 00:46:24 2018	(r329770)
> @@ -490,6 +490,7 @@ const TValue *luaH_get (Table *t, const TValue *key) {
>          return luaH_getint(t, k);  /* use specialized version */
>        /* else go through */

And again?

>      }
> +    /* FALLTHROUGH */
>      default: {
>        Node *n = mainposition(t, key);
>        do {  /* check whether `key' is somewhere in the chain */

-- 
Rod Grimes                                                 rgrimes at freebsd.org