Review Request: Fix return value in StreamReader::read
Kamil Klimek
naresh at tlen.pl
Mon Feb 21 13:26:38 CET 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100701/
-----------------------------------------------------------
Review request for Phonon and Phonon Backends.
Summary
-------
length parameter was treated as required size to read. It caused for files with id3v1 tags to end playback at begining of the file, because ret value was treated by libvlc as error. Reading id3v1 tags caused to seek almost to end of file and read not as much data as was put in length parameter. After returning length > 0 (we read at least something) everything started to work fine.
Diffs
-----
vlc/streamreader.cpp 89045da
Diff: http://git.reviewboard.kde.org/r/100701/diff
Testing
-------
Thanks,
Kamil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/phonon-backends/attachments/20110221/b0759483/attachment.htm
More information about the Phonon-backends
mailing list