[FFmpeg-devel] [PATCH] QCELP decoder
Reynaldo H. Verdejo Pinochet
reynaldo
Sat Nov 15 01:31:28 CET 2008
Hello
Kenan Gillet wrote:
>
> if it is not one of those, then we should have an I_F_Q.
> The SoC code determined the framerate by looking at the bufsize
> and then had a warning if the framerate byte (1st byte of the frame)
> differed.
> I suppose it was to handle frame which would not contain this first byte.
> I have not seen any files with such a feature though, and neither
> Reynaldo [1].
>
> We could simplify to just looking at the framerate and checking
> that the buffer contains enough data for the corresponding rate.
>
> what do you think?
>
> Renaldo, any thought on this one?
Well, I said this before but I will stress it further, my way
of thinking is stated in the code: My opinion is that if you
can, doing a little effort and at a reasonable cost, account
for an unexpected mistake then you should. that's what I did.
Bests
--
Reynaldo H. Verdejo Pinochet
More information about the ffmpeg-devel
mailing list