[FFmpeg-devel] camera files (no name)

[nicolas martin]

> On Wed, Feb 04, 2009 at 01:23:16PM -0500, nicolas martin wrote:
>>
>>> On Tue, Feb 03, 2009 at 04:27:51PM -0500, nicolas martin wrote:
>>>> Attached is a patch to support the nc4600 camera files.
>>>>
>>>> Thanks in advance for reviewing and replying.
>>>
>>> Changelog and docs update is missing.
>>
>> Doc update and changelog since when ?
>
> If you add a new demuxer, you should add an entry for it to the
> changelog and the format support list in doc/general.texi.
>

In reply to what Peter Ross wrote, I can't find its mail though

>> > + if (size <= 0) {
>> > + av_log(s, AV_LOG_DEBUG, "Wrong size : %d\n", size);
>> While size may be zero, it will never be less then zero, thus the  
>> debug statement is pointless.
The size can't be less than zero, I changed it.
However if the size is zero, then we have a problem, so the debug  
statement is meaningful.

>
> Diego
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patch_ncdec.txt
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090206/9cb60d06/attachment.txt>
-------------- next part --------------