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