[FFmpeg-devel] [PATCH] nuv: reject negative fps rate

[Michael Niedermayer]

On Thu, Dec 17, 2015 at 11:03:15PM +0100, Andreas Cadhalpun wrote:
> On 16.12.2015 23:53, Michael Niedermayer wrote:
> > On Wed, Dec 16, 2015 at 08:56:55PM +0100, Andreas Cadhalpun wrote:
> >> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
> >> ---
> >>  libavformat/nuv.c | 4 ++++
> >>  1 file changed, 4 insertions(+)
> > 
> > looks logical iam not nuv maintainer though and i dont know if all
> > files have fps set
> 
> I don't know that either, so I'm attaching a version of this patch
> that permits 0 fps. Which version do you prefer?

no real preferrance
i might slightly tend to allow fps == 0
but if 0 is disallowed and it occurs, people will tell us and we will
fix that
iam fine with either

theres no spec or reference that can be "checked" i assume ?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151218/ffb9e0f2/attachment.sig>