MJPEG Tools Svn

Don't need cpuinfo.c since it was used by cpuinfo.sh which has been removed.

If folks really want to play with "optimizing" for a specific cpu they can use
--with-extra-cflags="-mtune=xxxx" and try various values for xxxx

Remove cpuinfo.sh as both obsolete and not necessary. It turns out that
specifying -mcpu and -march makes no difference in the case of the encoder
(mpeg2enc). If giving the flags for one of the most cpu intense programs in
mjpegtools makes no difference then the cpuinfo.sh script should be removed.

Compiling mpeg2enc with -mtune=core2, corei7, generic and none (no flags at all)
and then encoding 10000 frames produced these times:

core2 118.318u 6.571s 2:05.61 99.4% 0+0k 5+43io 0pf+0w
corei7 118.831u 6.539s 2:05.63 99.7% 0+0k 2+24io 0pf+0w
generic 118.462u 6.571s 2:05.26 99.8% 0+0k 0+24io 0pf+0w
none 118.714u 6.597s 2:05.77 99.6% 0+0k 0+73io 0pf+0w

cpuinfo.sh has been removed from mjpegtools.

-tried to make this bitstreaming a bit quicker (~ 1% gained), but needs rewrite/ABI change to gain speed

- added subtitle stream as own enum
- clean up local variables
- bugfix: subtitles exeeding a pack won't segfault mplex any more

PROGRAM_NOPIC apparently causes problems on some x86_64 systems, has little to no performance effect on x86_64 systems and even on x86 systems most programs are I/O bound anyhow. So remove the 4 uses of the flag

No format string in sprintf - skip the sprintf completely.

Silence the warnings
ontheflyratectlpass2.cc:383:33: warning: '&&' within '||' [-Wlogical-op-parentheses]
if( actual_bits > target_bits && debiased_target_ABQ > picture.ABQ ||
~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~
ontheflyratectlpass2.cc:383:33: note: place parentheses around the '&&' expression to silence this warning

Fix the error
acroblock.cc:77:24: warning: & has lower precedence than ==; == will be evaluated first [-Wparentheses]
if( i->mb_type & MB_BACKWARD == 0 && cur_score < best_fwd_score)
^~~~~~~~~~~~~~~~~~
macroblock.cc:77:24: note: place parentheses around the '==' expression to silence this warning

usage function takes no arguments

why use a sprintf to copy a string - gives warning error format string is not a string literal (potentially insecure) [-Wformat-security]