Anders Waldenborg <[email protected]> writes:

>    Which part is the infinite loop fix?

The code in _flv_init that handles the data between the header
and tag data. The fix was changing a check for only a read error (-1) to that
and an unexpected EOS (0).

>
>    This would have been much easier to review if you did the
>    renaming of the constants in a separate commit?
>

Hm, I'm not one for splitting tiny changes into tinier changes but if it
makes it easier I don't see why not.

>

-- 
Please don't send me Word, Access, PowerPoint, or Excel attachments.
Ask for my public key. No sensitive data en clair.
9467 6082 8A35 2E1E 2D6B  76C4 5E9A ED56 076F 9E89

Attachment: pgpYWAvafDl4Q.pgp
Description: PGP signature

--
_______________________________________________
Xmms2-devel mailing list
[email protected]
http://lists.xmms.se/cgi-bin/mailman/listinfo/xmms2-devel

Reply via email to