hashirama/uvg266
1
0
Fork 0
mirror of https://github.com/ultravideo/uvg266.git synced 2024-12-02 21:24:07 +00:00
Code Issues Actions 1 Projects Releases Wiki Activity

Revert "Use sizeof(uint32_t) to avoid warning in GCC7."

Browse source 
Did not fix the problem.

This reverts commit e3c3e74926.
This commit is contained in:
Ari Lemmetti 2017-04-03 20:21:33 +03:00
parent e3c3e74926
commit 33ce101ab5
1 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/rdo.c
View file

@ -557,13 +557,11 @@ void kvz_rdoq(encoder_state_t * const state, coeff_t *coef, coeff_t *dest_coeff,
// Explicitly tell the only possible numbers of elements to be zeroed. // Explicitly tell the only possible numbers of elements to be zeroed.
// Hope the compiler is able to utilize this information. // Hope the compiler is able to utilize this information.
// TODO: Make sure this works if someone changes type of sig_coeffgoup_flag.
// Also check that eg. GCC7 does not warn about not multiplying by element size.
switch (cg_num) { switch (cg_num) {
case 1: memset(sig_coeffgroup_flag, 0, 1 * sizeof(uint32_t)); break; case 1: memset(sig_coeffgroup_flag, 0, 1 * sizeof(sig_coeffgroup_flag[0])); break;
case 4: memset(sig_coeffgroup_flag, 0, 4 * sizeof(uint32_t)); break; case 4: memset(sig_coeffgroup_flag, 0, 4 * sizeof(sig_coeffgroup_flag[0])); break;
case 16: memset(sig_coeffgroup_flag, 0, 16 * sizeof(uint32_t)); break; case 16: memset(sig_coeffgroup_flag, 0, 16 * sizeof(sig_coeffgroup_flag[0])); break;
case 64: memset(sig_coeffgroup_flag, 0, 64 * sizeof(uint32_t)); break; case 64: memset(sig_coeffgroup_flag, 0, 64 * sizeof(sig_coeffgroup_flag[0])); break;
default: assert(0 && "There should be 1, 4, 16 or 64 coefficient groups"); default: assert(0 && "There should be 1, 4, 16 or 64 coefficient groups");
} }