Re: [FFmpeg-devel] [PATCH] doc/codecs: Add missing documentation for apply_cropping

2017-05-10 Thread Michael Niedermayer 
On Tue, May 09, 2017 at 06:15:25PM -0800, Lou Logan wrote:
> On Tue,  9 May 2017 17:23:35 +0200
> Michael Niedermayer  wrote:
> 
> > Signed-off-by: Michael Niedermayer 
> > ---
> >  doc/codecs.texi | 10 ++
> >  1 file changed, 10 insertions(+)
> > 
> > diff --git a/doc/codecs.texi b/doc/codecs.texi
> > index 1f74c83554..1d8a4e38a4 100644
> > --- a/doc/codecs.texi
> > +++ b/doc/codecs.texi
> > @@ -1279,6 +1279,16 @@ ffprobe -dump_separator "
> >  Maximum number of pixels per image. This value can be used to avoid out of
> >  memory failures due to large images.
> >  
> > +@item apply_cropping @var{integer} (@emph{decoding,video})
> > +Enable cropping if cropping parameters are a multiply of the required
> 
> s/a multiply/multiples
> 
> > +alignment for the left and top parameters. If the alignment is not met the
> > +cropping will be partially applied to maintain alignment.
> > +1(Enabled) by default.
> 
> Default is 1 (enabled).
> 
> > +Note: The required alignment depends on if CODEC_FLAG_UNALIGNED is set and 
> > the
> 
> @code{CODEC_FLAG_UNALIGNED}
> 
> > +CPU. CODEC_FLAG_UNALIGNED cannot be chanaged from the command line. Also hw
> > +decoders will not apply left/top croping.
> 
> hardware

chanegs made

thx

[...9
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
On any real computer, always possible as a real computer has a finite number
of states N, and will either halt in less than N cycles or never halt.


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] doc/codecs: Add missing documentation for apply_cropping

2017-05-09 Thread Lou Logan 
On Tue,  9 May 2017 17:23:35 +0200
Michael Niedermayer  wrote:

> Signed-off-by: Michael Niedermayer 
> ---
>  doc/codecs.texi | 10 ++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/doc/codecs.texi b/doc/codecs.texi
> index 1f74c83554..1d8a4e38a4 100644
> --- a/doc/codecs.texi
> +++ b/doc/codecs.texi
> @@ -1279,6 +1279,16 @@ ffprobe -dump_separator "
>  Maximum number of pixels per image. This value can be used to avoid out of
>  memory failures due to large images.
>  
> +@item apply_cropping @var{integer} (@emph{decoding,video})
> +Enable cropping if cropping parameters are a multiply of the required

s/a multiply/multiples

> +alignment for the left and top parameters. If the alignment is not met the
> +cropping will be partially applied to maintain alignment.
> +1(Enabled) by default.

Default is 1 (enabled).

> +Note: The required alignment depends on if CODEC_FLAG_UNALIGNED is set and 
> the

@code{CODEC_FLAG_UNALIGNED}

> +CPU. CODEC_FLAG_UNALIGNED cannot be chanaged from the command line. Also hw
> +decoders will not apply left/top croping.

hardware
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] doc/codecs: Add missing documentation for apply_cropping

2017-05-09 Thread Michael Niedermayer 
On Tue, May 09, 2017 at 04:54:52PM -0300, James Almer wrote:
> On 5/9/2017 12:23 PM, Michael Niedermayer wrote:
> > Signed-off-by: Michael Niedermayer 
> > ---
> >  doc/codecs.texi | 10 ++
> >  1 file changed, 10 insertions(+)
> > 
> > diff --git a/doc/codecs.texi b/doc/codecs.texi
> > index 1f74c83554..1d8a4e38a4 100644
> > --- a/doc/codecs.texi
> > +++ b/doc/codecs.texi
> > @@ -1279,6 +1279,16 @@ ffprobe -dump_separator "
> >  Maximum number of pixels per image. This value can be used to avoid out of
> >  memory failures due to large images.
> >  
> > +@item apply_cropping @var{integer} (@emph{decoding,video})
> 
> This option (as well as others like skip_alpha, or refcounted_frames
> which is also missing in this file) are boolean, not int.
> In practice it makes no difference i guess, since the actual
> AVCodecContext fields are all int anyway, but maybe it could be
> reflected here. I'm not sure if the difference between AV_OPT_TYPE_BOOL
> and AV_OPT_TYPE_INT has any implications for API users, though.

will send a seperate patch for bool


> 
> > +Enable cropping if cropping parameters are a multiply of the required
> > +alignment for the left and top parameters. If the alignment is not met the
> > +cropping will be partially applied to maintain alignment.
> > +1(Enabled) by default.
> > +Note: The required alignment depends on if CODEC_FLAG_UNALIGNED is set and 
> > the
> > +CPU. CODEC_FLAG_UNALIGNED cannot be chanaged from the command line. Also hw
> 
> AV_CODEC_FLAG_UNALIGNED, and "changed".
> 
> > +decoders will not apply left/top croping.
> 
> Cropping.

changed applied

thx


[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] doc/codecs: Add missing documentation for apply_cropping

2017-05-09 Thread James Almer 
On 5/9/2017 12:23 PM, Michael Niedermayer wrote:
> Signed-off-by: Michael Niedermayer 
> ---
>  doc/codecs.texi | 10 ++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/doc/codecs.texi b/doc/codecs.texi
> index 1f74c83554..1d8a4e38a4 100644
> --- a/doc/codecs.texi
> +++ b/doc/codecs.texi
> @@ -1279,6 +1279,16 @@ ffprobe -dump_separator "
>  Maximum number of pixels per image. This value can be used to avoid out of
>  memory failures due to large images.
>  
> +@item apply_cropping @var{integer} (@emph{decoding,video})

This option (as well as others like skip_alpha, or refcounted_frames
which is also missing in this file) are boolean, not int.
In practice it makes no difference i guess, since the actual
AVCodecContext fields are all int anyway, but maybe it could be
reflected here. I'm not sure if the difference between AV_OPT_TYPE_BOOL
and AV_OPT_TYPE_INT has any implications for API users, though.

> +Enable cropping if cropping parameters are a multiply of the required
> +alignment for the left and top parameters. If the alignment is not met the
> +cropping will be partially applied to maintain alignment.
> +1(Enabled) by default.
> +Note: The required alignment depends on if CODEC_FLAG_UNALIGNED is set and 
> the
> +CPU. CODEC_FLAG_UNALIGNED cannot be chanaged from the command line. Also hw

AV_CODEC_FLAG_UNALIGNED, and "changed".

> +decoders will not apply left/top croping.

Cropping.

> +
> +
>  @end table
>  
>  @c man end CODEC OPTIONS
> 

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] doc/codecs: Add missing documentation for apply_cropping

2017-05-09 Thread Michael Niedermayer 
Signed-off-by: Michael Niedermayer 
---
 doc/codecs.texi | 10 ++
 1 file changed, 10 insertions(+)

diff --git a/doc/codecs.texi b/doc/codecs.texi
index 1f74c83554..1d8a4e38a4 100644
--- a/doc/codecs.texi
+++ b/doc/codecs.texi
@@ -1279,6 +1279,16 @@ ffprobe -dump_separator "
 Maximum number of pixels per image. This value can be used to avoid out of
 memory failures due to large images.
 
+@item apply_cropping @var{integer} (@emph{decoding,video})
+Enable cropping if cropping parameters are a multiply of the required
+alignment for the left and top parameters. If the alignment is not met the
+cropping will be partially applied to maintain alignment.
+1(Enabled) by default.
+Note: The required alignment depends on if CODEC_FLAG_UNALIGNED is set and the
+CPU. CODEC_FLAG_UNALIGNED cannot be chanaged from the command line. Also hw
+decoders will not apply left/top croping.
+
+
 @end table
 
 @c man end CODEC OPTIONS
-- 
2.11.0

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel