Menu

Commit [r3107]  Maximize  Restore  History

Enable some of the sanity checks that were hidden inside '#ifdef ALTIVEC_VERIFY'

conditionals. In particular the requirement in sad_00 that 'h' be 8 or 16 was
being violated by yuvdenoise and the program was mysteriously coredumping.

IF "user input" (in this case a parameter from a user program into a library)
can cause a program to segfault THEN that condition SHOULD BE CHECKED and
a meaninful error given.

After enabling the checks the cause of the problem was immediately known:

**ERROR: [yuvdenoise] sad_00: h != [8|16], (32)

sms00 2006-07-29

changed /trunk/mjpeg_play/utils/altivec/add_pred.c
changed /trunk/mjpeg_play/utils/altivec/bsad.c
changed /trunk/mjpeg_play/utils/altivec/bsumsq.c
changed /trunk/mjpeg_play/utils/altivec/bsumsq_sub22.c
changed /trunk/mjpeg_play/utils/altivec/build_sub22_mests.c
changed /trunk/mjpeg_play/utils/altivec/build_sub44_mests.c
changed /trunk/mjpeg_play/utils/altivec/find_best_one_pel.c
changed /trunk/mjpeg_play/utils/altivec/pred_comp.c
changed /trunk/mjpeg_play/utils/altivec/sad_00.c
changed /trunk/mjpeg_play/utils/altivec/sad_01.c
changed /trunk/mjpeg_play/utils/altivec/sad_10.c
changed /trunk/mjpeg_play/utils/altivec/sad_11.c
changed /trunk/mjpeg_play/utils/altivec/sub_pred.c
changed /trunk/mjpeg_play/utils/altivec/subsample_image.c
changed /trunk/mjpeg_play/utils/altivec/sumsq.c
changed /trunk/mjpeg_play/utils/altivec/sumsq_sub22.c
changed /trunk/mjpeg_play/utils/altivec/variance.c
/trunk/mjpeg_play/utils/altivec/add_pred.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/bsad.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/bsumsq.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/bsumsq_sub22.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/build_sub22_mests.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/build_sub44_mests.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/find_best_one_pel.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/pred_comp.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/sad_00.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/sad_01.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/sad_10.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/sad_11.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/sub_pred.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/subsample_image.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/sumsq.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/sumsq_sub22.c Diff Switch to side-by-side view
Loading...
/trunk/mjpeg_play/utils/altivec/variance.c Diff Switch to side-by-side view
Loading...
Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.