iodine 0.7.26 → 0.7.27

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cef74e69c89b287a19db0fdfe5dfe4d54df7e33026d786832aebb6e431457efa
-  data.tar.gz: 86a85ce79660e0df9bcd56b66a6ebd5d1274e29f83bffc1f004acf7bd99bbfe4
+  metadata.gz: 271062484b7941686926625d08f09463c7f723a94c9a43b1ef9a3b85c2b0e2e2
+  data.tar.gz: 7914c3a6e093c53bc2ff78ed1f6bc0f376d5dcb69bb2086187238fc9666894eb
 SHA512:
-  metadata.gz: b0d9c2b9730b802c9c085f820243ee20621ec845cde85411927a38661f903cf75792d143d163498bd5597c4301df38a881c123b604edee0bdfbe72b71bc2aab8
-  data.tar.gz: 16fd00d1387cf1aab22c58091abded717aca527b912dbdd7dc358eaec9d18a16b16919ad77177dc50d358c465c96af77dfa35433c95e1ad6ff67e585feb7b341
+  metadata.gz: 9ef504553715d08e8152e2a5195368965bdb71c9d6d52157a4796c58cb5533527008d29fa7fb03788df89f3a3c80db4210b4721b062291f0993529aae26f1347
+  data.tar.gz: f7afb8f4c3a0fbe19e396798f917d2c3d039d8ff318b43be2c059b73ec8cb079c80f1d4daa777aafc7206360aee2373d767c3e1089676ffbc70226c6b294681f

data/.gitignore

@@ -15,3 +15,5 @@
 .clang_complete
 
 .DS_Store
+spec/examples.txt
+spec/log

data/.rspec

@@ -0,0 +1,2 @@
+--require spec_helper
+--color

data/.travis.yml

@@ -20,20 +20,11 @@ addons:
     sources:
       - ubuntu-toolchain-r-test
     packages:
-      - gcc-4.9
-      - gcc-5
-      - clang
+      - gcc
 script:
   - echo CFLAGS = $CFLAGS
   - echo cflags = $cflags
   - gem uninstall -x iodine
   - rake build
   - find pkg/iodine-*.gem -exec gem install -V {} +
-  - gem uninstall -x iodine
-  - export CFLAGS="-std=c99"
-  - export CC=gcc
-  - find pkg/iodine-*.gem -exec gem install -V {} +
-  - gem uninstall -x iodine
-  - export CFLAGS="-Wall"
-  - export CC="gcc-5"
-  - find pkg/iodine-*.gem -exec gem install -V {} +
+  - bundle exec rspec --format documentation

data/CHANGELOG.md

@@ -6,6 +6,10 @@ Please notice that this change log contains changes for upcoming releases as wel
 
 ## Changes:
 
+#### Change log v.0.7.27
+
+**Compatibility**: (`iodine`) fixed the HTTP request `SCRIPT_NAME` variable (in the Rack `env`) to default to the global environment variable `SCRIPT_NAME` when `SCRIPT_NAME` isn't root (`/`). Credit to @thexa4 (Max Maton) for exposing this compatibility concern (issue #68).
+
 #### Change log v.0.7.26
 
 **Fix**: (`http`) fixed HTTP date format to force the day of the month to use two digits. Credit to @ianks (Ian Ker-Seymer) for exposing this issue (issue #64).

data/Gemfile

@@ -1,4 +1,11 @@
 source 'https://rubygems.org'
 
+group :test do
+  gem 'pry'
+  gem 'rspec'
+  gem 'rack'
+  gem 'http'
+end
+
 # Specify your gem's dependencies in iodine.gemspec
 gemspec

data/ext/iodine/fio.c

@@ -1397,17 +1397,20 @@ static void sig_int_handler(int sig) {
     old = &fio_old_sig_usr1;
     break;
 #endif
-  case SIGINT: /* fallthrough */
+    /* fallthrough */
+  case SIGINT:
     if (!old)
       old = &fio_old_sig_int;
-  case SIGTERM: /* fallthrough */
+    /* fallthrough */
+  case SIGTERM:
     if (!old)
       old = &fio_old_sig_term;
     fio_stop();
     break;
-  case SIGPIPE: /* fallthrough */
+  case SIGPIPE:
     if (!old)
       old = &fio_old_sig_pipe;
+  /* fallthrough */
   default:
     break;
   }
@@ -3005,23 +3008,23 @@ flush_rw_hook:
 test_errno:
   fio_unlock(&uuid_data(uuid).sock_lock);
   switch (errno) {
-  case EWOULDBLOCK: /* ovreflow */
+  case EWOULDBLOCK: /* fallthrough */
 #if EWOULDBLOCK != EAGAIN
-  case EAGAIN: /* ovreflow */
+  case EAGAIN: /* fallthrough */
 #endif
-  case ENOTCONN:      /* ovreflow */
-  case EINPROGRESS:   /* ovreflow */
-  case ENOSPC:        /* ovreflow */
-  case EADDRNOTAVAIL: /* ovreflow */
+  case ENOTCONN:      /* fallthrough */
+  case EINPROGRESS:   /* fallthrough */
+  case ENOSPC:        /* fallthrough */
+  case EADDRNOTAVAIL: /* fallthrough */
   case EINTR:
     return 1;
   case EFAULT:
     FIO_LOG_ERROR("fio_flush EFAULT - possible memory address error sent to "
                   "Unix socket.");
-    /* ovreflow */
-  case EPIPE:  /* ovreflow */
-  case EIO:    /* ovreflow */
-  case EINVAL: /* ovreflow */
+    /* fallthrough */
+  case EPIPE:  /* fallthrough */
+  case EIO:    /* fallthrough */
+  case EINVAL: /* fallthrough */
   case EBADF:
     uuid_data(uuid).close = 1;
     fio_force_close(uuid);
@@ -3362,13 +3365,13 @@ void fio_state_callback_force(callback_type_e c_type) {
   fio_lock(&callback_collection[c_type].lock);
   fio_state_callback_ensure(&callback_collection[c_type]);
   switch (c_type) {            /* the difference between `unshift` and `push` */
-  case FIO_CALL_ON_INITIALIZE: /* overflow */
-  case FIO_CALL_PRE_START:     /* overflow */
-  case FIO_CALL_BEFORE_FORK:   /* overflow */
-  case FIO_CALL_AFTER_FORK:    /* overflow */
-  case FIO_CALL_IN_CHILD:      /* overflow */
-  case FIO_CALL_IN_MASTER:     /* overflow */
-  case FIO_CALL_ON_START:      /* overflow */
+  case FIO_CALL_ON_INITIALIZE: /* fallthrough */
+  case FIO_CALL_PRE_START:     /* fallthrough */
+  case FIO_CALL_BEFORE_FORK:   /* fallthrough */
+  case FIO_CALL_AFTER_FORK:    /* fallthrough */
+  case FIO_CALL_IN_CHILD:      /* fallthrough */
+  case FIO_CALL_IN_MASTER:     /* fallthrough */
+  case FIO_CALL_ON_START:      /* fallthrough */
     FIO_LS_EMBD_FOR(&callback_collection[c_type].callbacks, pos) {
       callback_data_s *tmp = fio_malloc(sizeof(*tmp));
       FIO_ASSERT_ALLOC(tmp);
@@ -3385,12 +3388,12 @@ void fio_state_callback_force(callback_type_e c_type) {
     }
     break;
 
-  case FIO_CALL_ON_SHUTDOWN:     /* overflow */
-  case FIO_CALL_ON_FINISH:       /* overflow */
-  case FIO_CALL_ON_PARENT_CRUSH: /* overflow */
-  case FIO_CALL_ON_CHILD_CRUSH:  /* overflow */
-  case FIO_CALL_AT_EXIT:         /* overflow */
-  case FIO_CALL_NEVER:           /* overflow */
+  case FIO_CALL_ON_SHUTDOWN:     /* fallthrough */
+  case FIO_CALL_ON_FINISH:       /* fallthrough */
+  case FIO_CALL_ON_PARENT_CRUSH: /* fallthrough */
+  case FIO_CALL_ON_CHILD_CRUSH:  /* fallthrough */
+  case FIO_CALL_AT_EXIT:         /* fallthrough */
+  case FIO_CALL_NEVER:           /* fallthrough */
   default:
     FIO_LS_EMBD_FOR(&callback_collection[c_type].callbacks, pos) {
       callback_data_s *tmp = fio_malloc(sizeof(*tmp));
@@ -4148,7 +4151,7 @@ size_t fio_ltoa(char *dest, int64_t num, uint8_t base) {
     goto zero;
 
   switch (base) {
-  case 1:
+  case 1: /* fallthrough */
   case 2:
     /* Base 2 */
     {
@@ -4224,12 +4227,12 @@ size_t fio_ltoa(char *dest, int64_t num, uint8_t base) {
       dest[len] = 0;
       return len;
     }
-  case 3:
-  case 4:
-  case 5:
-  case 6:
-  case 7:
-  case 9:
+  case 3: /* fallthrough */
+  case 4: /* fallthrough */
+  case 5: /* fallthrough */
+  case 6: /* fallthrough */
+  case 7: /* fallthrough */
+  case 9: /* fallthrough */
     /* rare bases */
     if (num < 0) {
       dest[len++] = '-';
@@ -6655,7 +6658,7 @@ static int fio_glob_match(fio_str_info_s pat, fio_str_info_s ch) {
     case '\\':
       d = *(uint8_t *)pat.data++;
       pat.len--;
-    /* FALLTHROUGH */
+    /* fallthrough */
     default: /* Literal character */
       if (c == d)
         break;
@@ -7494,10 +7497,12 @@ void fio_rand_bytes(void *data_, size_t len) {
   case 24:
     tmp = fio_rand64();
     fio_u2str64(data + 16, tmp);
-  case 16: /* overflow */
+    /* fallthrough */
+  case 16:
     tmp = fio_rand64();
     fio_u2str64(data + 8, tmp);
-  case 8: /* overflow */
+    /* fallthrough */
+  case 8:
     tmp = fio_rand64();
     fio_u2str64(data, tmp);
     data += len & 24;
@@ -7506,19 +7511,25 @@ void fio_rand_bytes(void *data_, size_t len) {
     tmp = fio_rand64();
     /* leftover bytes */
     switch ((len & 7)) {
-    case 7: /* overflow */
+    case 7:
       data[6] = (tmp >> 8) & 0xFF;
-    case 6: /* overflow */
+      /* fallthrough */
+    case 6:
       data[5] = (tmp >> 16) & 0xFF;
-    case 5: /* overflow */
+      /* fallthrough */
+    case 5:
       data[4] = (tmp >> 24) & 0xFF;
-    case 4: /* overflow */
+      /* fallthrough */
+    case 4:
       data[3] = (tmp >> 32) & 0xFF;
-    case 3: /* overflow */
+      /* fallthrough */
+    case 3:
       data[2] = (tmp >> 40) & 0xFF;
-    case 2: /* overflow */
+      /* fallthrough */
+    case 2:
       data[1] = (tmp >> 48) & 0xFF;
-    case 1: /* overflow */
+      /* fallthrough */
+    case 1:
       data[0] = (tmp >> 56) & 0xFF;
     }
   }

data/ext/iodine/fio.h

@@ -2288,14 +2288,14 @@ FIO_FUNC inline uintptr_t fio_ct_if2(uintptr_t cond, uintptr_t a, uintptr_t b) {
 /** Writes a local 64 bit number to an unaligned buffer in network order. */
 #define fio_u2str64(buffer, i)                                                 \
   do {                                                                         \
-    ((uint8_t *)(buffer))[0] = ((uint64_t)(i) >> 56) & 0xFF;                   \
-    ((uint8_t *)(buffer))[1] = ((uint64_t)(i) >> 48) & 0xFF;                   \
-    ((uint8_t *)(buffer))[2] = ((uint64_t)(i) >> 40) & 0xFF;                   \
-    ((uint8_t *)(buffer))[3] = ((uint64_t)(i) >> 32) & 0xFF;                   \
-    ((uint8_t *)(buffer))[4] = ((uint64_t)(i) >> 24) & 0xFF;                   \
-    ((uint8_t *)(buffer))[5] = ((uint64_t)(i) >> 16) & 0xFF;                   \
-    ((uint8_t *)(buffer))[6] = ((uint64_t)(i) >> 8) & 0xFF;                    \
-    ((uint8_t *)(buffer))[7] = ((uint64_t)(i)) & 0xFF;                         \
+    ((uint8_t *)(buffer))[0] = (((uint64_t)(i) >> 56) & 0xFF);                 \
+    ((uint8_t *)(buffer))[1] = (((uint64_t)(i) >> 48) & 0xFF);                 \
+    ((uint8_t *)(buffer))[2] = (((uint64_t)(i) >> 40) & 0xFF);                 \
+    ((uint8_t *)(buffer))[3] = (((uint64_t)(i) >> 32) & 0xFF);                 \
+    ((uint8_t *)(buffer))[4] = (((uint64_t)(i) >> 24) & 0xFF);                 \
+    ((uint8_t *)(buffer))[5] = (((uint64_t)(i) >> 16) & 0xFF);                 \
+    ((uint8_t *)(buffer))[6] = (((uint64_t)(i) >> 8) & 0xFF);                  \
+    ((uint8_t *)(buffer))[7] = (((uint64_t)(i)) & 0xFF);                       \
   } while (0);
 
 /* *****************************************************************************
@@ -2478,9 +2478,11 @@ FIO_FUNC inline uint64_t fio_risky_hash(const void *data_, size_t len,
   switch (len & 24) {
   case 24:
     fio_risky_consume(v2, fio_str2u64(data + 16));
-  case 16: /* overflow */
+    /* fallthrough */
+  case 16:
     fio_risky_consume(v1, fio_str2u64(data + 8));
-  case 8: /* overflow */
+    /* fallthrough */
+  case 8:
     fio_risky_consume(v0, fio_str2u64(data));
     data += len & 24;
   }
@@ -2488,19 +2490,25 @@ FIO_FUNC inline uint64_t fio_risky_hash(const void *data_, size_t len,
   uint64_t tmp = 0;
   /* consume leftover bytes, if any */
   switch ((len & 7)) {
-  case 7: /* overflow */
+  case 7:
     tmp |= ((uint64_t)data[6]) << 8;
-  case 6: /* overflow */
+    /* fallthrough */
+  case 6:
     tmp |= ((uint64_t)data[5]) << 16;
-  case 5: /* overflow */
+    /* fallthrough */
+  case 5:
     tmp |= ((uint64_t)data[4]) << 24;
-  case 4: /* overflow */
+    /* fallthrough */
+  case 4:
     tmp |= ((uint64_t)data[3]) << 32;
-  case 3: /* overflow */
+    /* fallthrough */
+  case 3:
     tmp |= ((uint64_t)data[2]) << 40;
-  case 2: /* overflow */
+    /* fallthrough */
+  case 2:
     tmp |= ((uint64_t)data[1]) << 48;
-  case 1: /* overflow */
+    /* fallthrough */
+  case 1:
     tmp |= ((uint64_t)data[0]) << 56;
     /* ((len >> 3) & 3) is a 0...3 value indicating consumption vector */
     switch ((len >> 3) & 3) {

data/ext/iodine/fio_cli.c

@@ -156,7 +156,8 @@ static void fio_cli_set_arg(cstr_s arg, char const *value, char const *line,
         goto error;
       }
     }
-  case FIO_CLI_STRING__TYPE_I: /* fallthrough */
+    /* fallthrough */
+  case FIO_CLI_STRING__TYPE_I:
     if (!value)
       goto error;
     if (!value[0])
@@ -202,11 +203,11 @@ print_help:
   char const **pos = parser->names;
   while (*pos) {
     switch ((intptr_t)*pos) {
-    case FIO_CLI_STRING__TYPE_I:       /* fallthrough */
-    case FIO_CLI_BOOL__TYPE_I:         /* fallthrough */
-    case FIO_CLI_INT__TYPE_I:          /* fallthrough */
-    case FIO_CLI_PRINT__TYPE_I:        /* fallthrough */
-    case FIO_CLI_PRINT_HEADER__TYPE_I: /* fallthrough */
+    case FIO_CLI_STRING__TYPE_I: /* fallthrough */
+    case FIO_CLI_BOOL__TYPE_I:   /* fallthrough */
+    case FIO_CLI_INT__TYPE_I:    /* fallthrough */
+    case FIO_CLI_PRINT__TYPE_I:  /* fallthrough */
+    case FIO_CLI_PRINT_HEADER__TYPE_I:
       ++pos;
       continue;
     }

data/ext/iodine/fiobject.c

@@ -108,7 +108,7 @@ size_t __attribute__((weak)) fio_ltoa(char *dest, int64_t num, uint8_t base) {
     goto zero;
 
   switch (base) {
-  case 1:
+  case 1: /* fallthrough */
   case 2:
     /* Base 2 */
     {
@@ -184,12 +184,12 @@ size_t __attribute__((weak)) fio_ltoa(char *dest, int64_t num, uint8_t base) {
       dest[len] = 0;
       return len;
     }
-  case 3:
-  case 4:
-  case 5:
-  case 6:
-  case 7:
-  case 9:
+  case 3: /* fallthrough */
+  case 4: /* fallthrough */
+  case 5: /* fallthrough */
+  case 6: /* fallthrough */
+  case 7: /* fallthrough */
+  case 9: /* fallthrough */
     /* rare bases */
     if (num < 0) {
       dest[len++] = '-';
@@ -232,10 +232,11 @@ size_t __attribute__((weak)) fio_ltoa(char *dest, int64_t num, uint8_t base) {
 
 zero:
   switch (base) {
-  case 1:
+  case 1: /* fallthrough */
   case 2:
     dest[len++] = '0';
     dest[len++] = 'b';
+    /* fallthrough */
   case 16:
     dest[len++] = '0';
     dest[len++] = 'x';

data/ext/iodine/hpack.h

@@ -113,14 +113,14 @@ typedef const struct {
   const uint32_t code;
   const uint8_t bits;
 } huffman_encode_s;
-static const huffman_encode_s huffman_encode_table[];
+static const huffman_encode_s huffman_encode_table[257];
 
 /* the huffman decoding binary tree type */
 typedef struct {
   const int16_t value;     // value, -1 == none.
   const uint8_t offset[2]; // offset for 0 and one. 0 == leaf node.
 } huffman_decode_s;
-static const huffman_decode_s huffman_decode_tree[];
+static const huffman_decode_s huffman_decode_tree[513];
 
 /**
  * Unpack (de-compress) using HPACK huffman - returns the number of bytes
@@ -388,11 +388,11 @@ static MAYBE_UNUSED int hpack_huffman_pack(void *dest_, const int limit,
     switch (bits >> 3) {
     case 3:
       dest[comp_len + 2] = (uint8_t)(code >> 8) & 0xFF;
-    /* overflow */
+    /* fallthrough */
     case 2:
       dest[comp_len + 1] = (uint8_t)(code >> 16) & 0xFF;
-    /* overflow */
-    case 1: /* overflow */
+    /* fallthrough */
+    case 1:
       dest[comp_len + 0] = (uint8_t)(code >> 24) & 0xFF;
       comp_len += (bits >> 3);
       code <<= (bits & (~7));
@@ -426,7 +426,7 @@ calc_final_length:
 Header static table lookup
 ***************************************************************************** */
 
-const static struct {
+static const struct {
   struct hpack_static_data_s {
     const char *val;
     const size_t len;
@@ -1058,10 +1058,11 @@ void huffman__print_tree(void) {
       }
     }
   }
-  fprintf(stderr, "***** Copy after this line ****\n\n"
-                  "/** Static Huffman encoding map, left aligned */\n"
+  fprintf(stderr,
+          "***** Copy after this line ****\n\n"
+          "/** Static Huffman encoding map, left aligned */\n"
 
-                  "static const huffman_encode_s huffman_encode_table[] = {\n");
+          "static const huffman_encode_s huffman_encode_table[257] = {\n");
   for (size_t i = 0; i < 257; ++i) {
     /* print huffman code left align */
     fprintf(stderr, " {.code = 0x%.08X, .bits = %u}, // [%zu] \n",
@@ -1071,7 +1072,8 @@ void huffman__print_tree(void) {
   fprintf(stderr,
           "};\n\n/** Static Huffman decoding tree, flattened as an array */\n"
 
-          "static const huffman_decode_s huffman_decode_tree[] = {\n");
+          "static const huffman_decode_s huffman_decode_tree[%zu] = {\n",
+          tree_len);
   for (size_t i = 0; i < tree_len; ++i) {
     huffman__print_unit(
         tree[i], i,
@@ -1127,7 +1129,7 @@ int main(void) {
 ***************************************************************************** */
 
 /** Static Huffman encoding map, left aligned */
-static const huffman_encode_s huffman_encode_table[] = {
+static const huffman_encode_s huffman_encode_table[257] = {
     {.code = 0xFFC00000, .bits = 13}, // [0]
     {.code = 0xFFFFB000, .bits = 23}, // [1]
     {.code = 0xFFFFFE20, .bits = 28}, // [2]
@@ -1388,7 +1390,7 @@ static const huffman_encode_s huffman_encode_table[] = {
 };
 
 /** Static Huffman decoding tree, flattened as an array */
-static const huffman_decode_s huffman_decode_tree[] = {
+static const huffman_decode_s huffman_decode_tree[513] = {
     {.value = -1, .offset = {1, 44}}, // [0]
     {.value = -1, .offset = {1, 16}}, // [1]
     {.value = -1, .offset = {1, 8}},  // [2]

data/ext/iodine/http.c

@@ -2191,7 +2191,7 @@ static const char *MONTH_NAMES[] = {"Jan ", "Feb ", "Mar ", "Apr ",
                                     "Sep ", "Oct ", "Nov ", "Dec "};
 static const char *GMT_STR = "GMT";
 
-size_t http_date2str(char *target, struct tm *tmbuf) {
+size_t http_date2rfc7231(char *target, struct tm *tmbuf) {
   /* note: day of month is always 2 digits */
   char *pos = target;
   uint16_t tmp;
@@ -2235,6 +2235,8 @@ size_t http_date2str(char *target, struct tm *tmbuf) {
   return pos - target;
 }
 
+size_t http_date2str(char *target, struct tm *tmbuf);
+
 size_t http_date2rfc2822(char *target, struct tm *tmbuf) {
   /* note: day of month is either 1 or 2 digits */
   char *pos = target;

data/ext/iodine/http.h

@@ -905,6 +905,12 @@ Falls back to `gmtime_r` for dates before epoch.
 */
 struct tm *http_gmtime(time_t timer, struct tm *tmbuf);
 
+/** Writes an RFC 7231 date representation (HTTP date format) to target. */
+size_t http_date2rfc7231(char *target, struct tm *tmbuf);
+/** Writes an RFC 2109 date representation to target. */
+size_t http_date2rfc2109(char *target, struct tm *tmbuf);
+/** Writes an RFC 2822 date representation to target. */
+size_t http_date2rfc2822(char *target, struct tm *tmbuf);
 /**
 Writes an HTTP date string to the `target` buffer.
 
@@ -913,11 +919,9 @@ it's a super funky year, 32 bytes is about 3 more than you need).
 
 Returns the number of bytes actually written.
 */
-size_t http_date2str(char *target, struct tm *tmbuf);
-/** An alternative, RFC 2109 date representation. Requires */
-size_t http_date2rfc2109(char *target, struct tm *tmbuf);
-/** An alternative, RFC 2822 date representation. */
-size_t http_date2rfc2822(char *target, struct tm *tmbuf);
+static inline size_t http_date2str(char *target, struct tm *tmbuf) {
+  return http_date2rfc7231(target, tmbuf);
+}
 
 /**
  * Prints Unix time to a HTTP time formatted string.

data/ext/iodine/iodine_http.c

@@ -834,7 +834,14 @@ static void initialize_env_template(void) {
     }
     add_value_to_env(env_template_no_upgrade, "rack.version", rack_version);
   }
-  add_str_to_env(env_template_no_upgrade, "SCRIPT_NAME", "");
+
+  {
+    const char *sn = getenv("SCRIPT_NAME");
+    if (!sn || (sn[0] == '/' && sn[1] == 0)) {
+      sn = "";
+    }
+    add_str_to_env(env_template_no_upgrade, "SCRIPT_NAME", sn);
+  }
   add_value_to_env(env_template_no_upgrade, "rack.errors", rb_stderr);
   add_value_to_env(env_template_no_upgrade, "rack.hijack?", Qtrue);
   add_value_to_env(env_template_no_upgrade, "rack.multiprocess", Qtrue);

data/ext/iodine/mustache_parser.h

@@ -1046,7 +1046,8 @@ MUSTACHE_FUNC int(mustache_build)(mustache_build_args_s args) {
                            instructions[s.pos].data.name_len))
         goto user_error;
       break;
-    case MUSTACHE_WRITE_ARG: /* overflow */
+      /* fallthrough */
+    case MUSTACHE_WRITE_ARG:
       if (mustache_on_arg(&s.stack[s.index].sec,
                           data + instructions[s.pos].data.name_pos,
                           instructions[s.pos].data.name_len, 1))
@@ -1058,8 +1059,10 @@ MUSTACHE_FUNC int(mustache_build)(mustache_build_args_s args) {
                           instructions[s.pos].data.name_len, 0))
         goto user_error;
       break;
-    case MUSTACHE_SECTION_GOTO:  /* overflow */
-    case MUSTACHE_SECTION_START: /* overflow */
+      /* fallthrough */
+    case MUSTACHE_SECTION_GOTO:
+    /* fallthrough */
+    case MUSTACHE_SECTION_START:
     case MUSTACHE_SECTION_START_INV:
       /* advance stack*/
       if (s.index + 1 >= MUSTACHE_NESTING_LIMIT) {
@@ -1094,8 +1097,7 @@ MUSTACHE_FUNC int(mustache_build)(mustache_build_args_s args) {
         }
         s.stack[s.index].count = (uint32_t)val;
       }
-
-    /* overflow  */
+      /* fallthrough  */
     case MUSTACHE_SECTION_END:
       /* loop section or continue */
       if (s.stack[s.index].index < s.stack[s.index].count) {
@@ -1310,9 +1312,10 @@ MUSTACHE_FUNC mustache_s *(mustache_load)(mustache_load_args_s args) {
         }
         break;
 
-      case '^': /*overflow*/
+      case '^':
         /* start inverted section */
         flag = 0;
+        /* fallthrough */
       case '#':
         /* start section (or inverted section) */
         mustache__stand_alone_adjust(&s, stand_alone);
@@ -1449,14 +1452,14 @@ MUSTACHE_FUNC mustache_s *(mustache_load)(mustache_load_args_s args) {
             s.stack[s.index].del_end[s.stack[s.index].del_end_len - 1] == '}') {
           ++s.stack[s.index].data_pos;
         }
-        /*overflow*/
-      case '&': /*overflow*/
+        /* fallthrough */
+      case '&':
         /* unescaped variable data */
         flag = 0;
-        /* overflow to default */
-      case ':': /*overflow*/
-      case '<': /*overflow*/
-        ++beg;  /*overflow*/
+        /* fallthrough */
+      case ':': /*fallthrough*/
+      case '<': /*fallthrough*/
+        ++beg;  /*fallthrough*/
       default:
         --end;
         MUSTACHE_IGNORE_WHITESPACE(beg, 1);

data/lib/iodine/version.rb

@@ -1,3 +1,3 @@
 module Iodine
-  VERSION = '0.7.26'.freeze
+  VERSION = '0.7.27'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: iodine
 version: !ruby/object:Gem::Version
-  version: 0.7.26
+  version: 0.7.27
 platform: ruby
 authors:
 - Boaz Segev
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-03-07 00:00:00.000000000 Z
+date: 2019-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -82,6 +82,7 @@ extensions:
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - ".travis.yml"
 - ".yardopts"
 - CHANGELOG.md
@@ -201,7 +202,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 'Thank you for installing Iodine 0.7.26.
+post_install_message: 'Thank you for installing Iodine 0.7.27.
 
 '
 rdoc_options: []