Fix #197: Missing braces in for loop

[Zenexer]

Fixes #197.

[fduncanh]

Those braces are optional, it's a one line "for" statement. Your proposed change is cosmetic.

[Zenexer]

@fduncanh They aren't optional if that entire line is meant to be looped. Currently, it's effectively this:

for ( i = 0; i < s->payloadSize; i++ ) {
    printf("%ld.%d: ", (long int)(b->p - b->start), b->bits_left);
}

s->data[i] = bs_read_u8(b); printf("s->data[i]: %d \n", s->data[i]); 

Implicit braces are per-statement, not per-line.

[fduncanh]

sorry, you are right of course. Luckily, from the function name it seems to be in code only used at some time for debugging h264-bitstream code.

It's never referenced unless HAVE_SEI is defined, which appears not to be the case. (so it appears to be dead code)

In my use (not RPi) of RPiPlay, that h264-bitstream code is never compiled,

[fduncanh]

you should report it upstream here: (there appears still to be someone occasionally looking at issues there)

https://github.com/aizvorski/h264bitstream

[FD-]

Yeah, please report it to upstream! We're not using that part of the code, but other consumers of h264bitstream may.