static analysis pickmodes issues

GoogleCodeExporter commented 9 years ago

Deb, I think these are your additions but if I'm wrong feel free to assign to 

    [CC] vp9/encoder/vp9_pickmode.c.o
/usr/local/google/vpnext/vp9/encoder/vp9_pickmode.c:91:5: warning: Value stored 
to 'bestsme' is never read
    bestsme = vp9_fast_hex_search(x, &mvp_full, step_param, sadpb, 0,
    ^         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/local/google/vpnext/vp9/encoder/vp9_pickmode.c:96:5: warning: Value stored 
to 'bestsme' is never read
    bestsme = vp9_hex_search(x, &mvp_full, step_param, sadpb, 1,
    ^         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/local/google/vpnext/vp9/encoder/vp9_pickmode.c:101:5: warning: Value 
stored to 'bestsme' is never read
    bestsme = vp9_square_search(x, &mvp_full, step_param, sadpb, 1,
    ^         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/local/google/vpnext/vp9/encoder/vp9_pickmode.c:106:5: warning: Value 
stored to 'bestsme' is never read
    bestsme = vp9_bigdia_search(x, &mvp_full, step_param, sadpb, 1,
    ^         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/local/google/vpnext/vp9/encoder/vp9_pickmode.c:112:5: warning: Value 
stored to 'bestsme' is never read
    bestsme = vp9_full_pixel_diamond(cpi, x, &mvp_full, step_param,
    ^         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Original issue reported on code.google.com by jimbankoski@google.com on 6 Mar 2014 at 3:36

GoogleCodeExporter commented 9 years ago

https://gerrit.chromium.org/gerrit/#/c/69135/

Original comment by debar...@google.com on 6 Mar 2014 at 7:08

GoogleCodeExporter commented 9 years ago

Should be already fixed.

Original comment by dkova...@google.com on 27 May 2014 at 7:17

Changed state: Fixed

ericmckean / webm

static analysis pickmodes issues #722