Better Huffman2 implementation

[nooneimprt4nt@…]

This is an improved Huffman2 decoding implementation. It uses much less CPU load than the current implementation, which is actually noticable (~70% CPU usage down to ~50%) when processing the EIT from my provider.

[anonymous]

The patch applied cleanly, but I get the following error, compiling

In file included from siparser.cpp:15:
atsc_huffman.h:638: error: syntax error before `[' token
atsc_huffman.h:768: error: syntax error before `[' token
siparser.cpp: In member function `void SIParser::ParseDVBEIT(unsigned int,
   tablehead_t*, uint8_t*, unsigned int)':
siparser.cpp:1513: warning: comparison between signed and unsigned integer
   expressions
siparser.cpp: In member function `int SIParser::Huffman2ToQString(uint8_t*,
   int, int, QString&)':
siparser.cpp:2838: error: `Huff2Lookup128' undeclared (first use this function)
siparser.cpp:2838: error: (Each undeclared identifier is reported only once for
   each function it appears in.)
siparser.cpp:2845: error: `Huff2Lookup256' undeclared (first use this function)
make[2]: *** [siparser.o] Error 1
make[2]: Leaving directory `/root/mythtv/trunk/mythtv/libs/libmythtv'
make[1]: *** [sub-libmythtv] Error 2
make[1]: Leaving directory `/root/mythtv/trunk/mythtv/libs'
make: *** [sub-libs] Error 2

[nooneimprt4nt@…]

maybe uint8_t isn't defined at the point when atsc_huffman.h is loaded?

can you try swapping the uint8_t with 'unsigned char' in atsc_huffman.h. If that works, I'll update the patch, though I have no idea why it doesn't work for you, compiles without warnings on my gcc 4.03 system

[anonymous]

changing to unsigned char allowed the compile to complete

thanks

[nooneimprt4nt@…]

change uint8_t to unsigned char for compile issue

[danielk]

(In [8943]) Closes #1094, by applying modified patch.

This changes to a faster huffman decoding algorithm for ATSC EIT data.

I also added some const correctness and formatting fixes to the patch, mostly to make it easier to move these functions as part of #1035.