Review Request: Fix return value in StreamReader::read

Kamil Klimek naresh at tlen.pl
Mon Feb 21 12:26:38 GMT 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/kde-multimedia/attachments/20110221/b0759483/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-multimedia mailing list
kde-multimedia at kde.org
https://mail.kde.org/mailman/listinfo/kde-multimedia


More information about the kde-multimedia mailing list