Skip to content

Commit

Permalink
Re-enable the assert (#477)
Browse files Browse the repository at this point in the history
The assert in fasthash_accum_cstring was previously disabled due to
crash during restore. Re-enable this assert since the related testcase has
been updated to fix the issue.

Task: BABEL-5421
Signed-off-by: Rishabh Tanwar ritanwar@amazon.com
  • Loading branch information
rishabhtanwar29 authored Nov 25, 2024
1 parent 1e4f6fc commit 6a6632a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/include/common/hashfn_unstable.h
Original file line number Diff line number Diff line change
Expand Up @@ -348,7 +348,7 @@ fasthash_accum_cstring(fasthash_state *hs, const char *str)
{
len = fasthash_accum_cstring_aligned(hs, str);
Assert(len_check == len);
// TODO: Assert(hs_check.hash == hs->hash);
Assert(hs_check.hash == hs->hash);
return len;
}
#endif /* SIZEOF_VOID_P */
Expand Down

0 comments on commit 6a6632a

Please sign in to comment.