commit 709c4672d4a387f56839e47ed6a985cd2f950a8f
author Brotli <no-reply@google.com> Fri Apr 12 09:15:22 2024 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Apr 12 09:16:00 2024 -0700
tree 30dd6c769312d5cfd046d8e8085de5a95e18d7c9
parent a76d96e7308614b422508679d6905cbe1a8762bf

Fix minor syntax issues.

Missing semicolons.
Move checks below variable declarations for c89.

PiperOrigin-RevId: 624199887

c/enc/hash.h

diff --git a/c/enc/hash.h b/c/enc/hash.h
index f3a915b..d993b6a 100644
--- a/c/enc/hash.h
+++ b/c/enc/hash.h
@@ -557,7 +557,7 @@
     offset = distance_offset - distance;
     limit = source_size - offset;
     limit = limit > max_length ? max_length : limit;
-    BROTLI_DCHECK(cur_ix_masked + limit <= ring_buffer_mask)
+    BROTLI_DCHECK(cur_ix_masked + limit <= ring_buffer_mask);
     len = FindMatchLengthWithLimit(&source[offset], &data[cur_ix_masked],
                                    limit);
     if (len >= 2) {

c/enc/hash_forgetful_chain_inc.h

diff --git a/c/enc/hash_forgetful_chain_inc.h b/c/enc/hash_forgetful_chain_inc.h
index 17f31fa..0e28b7e 100644
--- a/c/enc/hash_forgetful_chain_inc.h
+++ b/c/enc/hash_forgetful_chain_inc.h
@@ -203,12 +203,12 @@
   uint8_t* BROTLI_RESTRICT tiny_hashes = FN(TinyHash)(self->extra[0]);
   FN(Bank)* BROTLI_RESTRICT banks = FN(Banks)(self->extra[1]);
   const size_t cur_ix_masked = cur_ix & ring_buffer_mask;
-  BROTLI_DCHECK(cur_ix_masked + max_length <= ring_buffer_mask)
   /* Don't accept a short copy from far away. */
   score_t min_score = out->score;
   score_t best_score = out->score;
   size_t best_len = out->len;
   size_t i;
+  BROTLI_DCHECK(cur_ix_masked + max_length <= ring_buffer_mask);
   const size_t key = FN(HashBytes)(&data[cur_ix_masked]);
   const uint8_t tiny_hash = (uint8_t)(key);
   out->len = 0;

c/enc/hash_longest_match64_inc.h

diff --git a/c/enc/hash_longest_match64_inc.h b/c/enc/hash_longest_match64_inc.h
index 9834c2a..62d51f6 100644
--- a/c/enc/hash_longest_match64_inc.h
+++ b/c/enc/hash_longest_match64_inc.h
@@ -165,12 +165,12 @@
   uint16_t* BROTLI_RESTRICT num = self->num_;
   uint32_t* BROTLI_RESTRICT buckets = self->buckets_;
   const size_t cur_ix_masked = cur_ix & ring_buffer_mask;
-  BROTLI_DCHECK(cur_ix_masked + max_length <= ring_buffer_mask)
   /* Don't accept a short copy from far away. */
   score_t min_score = out->score;
   score_t best_score = out->score;
   size_t best_len = out->len;
   size_t i;
+  BROTLI_DCHECK(cur_ix_masked + max_length <= ring_buffer_mask);
   out->len = 0;
   out->len_code_delta = 0;
   /* Try last distance first. */

c/enc/hash_longest_match_inc.h

diff --git a/c/enc/hash_longest_match_inc.h b/c/enc/hash_longest_match_inc.h
index 42b9532..6de9ba5 100644
--- a/c/enc/hash_longest_match_inc.h
+++ b/c/enc/hash_longest_match_inc.h
@@ -164,12 +164,12 @@
   uint16_t* BROTLI_RESTRICT num = self->num_;
   uint32_t* BROTLI_RESTRICT buckets = self->buckets_;
   const size_t cur_ix_masked = cur_ix & ring_buffer_mask;
-  BROTLI_DCHECK(cur_ix_masked + max_length <= ring_buffer_mask)
   /* Don't accept a short copy from far away. */
   score_t min_score = out->score;
   score_t best_score = out->score;
   size_t best_len = out->len;
   size_t i;
+  BROTLI_DCHECK(cur_ix_masked + max_length <= ring_buffer_mask);
   out->len = 0;
   out->len_code_delta = 0;
   /* Try last distance first. */