On Thu, Jan 22, 2015 at 02:44:55PM +, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis
> ---
> libavcodec/libx265.c | 5 +
> 1 file changed, 5 insertions(+)
LGTM
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
What does censorship reveal? It r
On 1/22/2015 2:44 PM, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis
> ---
> libavcodec/libx265.c | 5 +
> 1 file changed, 5 insertions(+)
This is v2.
- Derek
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailm
Signed-off-by: Derek Buitenhuis
---
libavcodec/libx265.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c
index 923c750..aee7ae2 100644
--- a/libavcodec/libx265.c
+++ b/libavcodec/libx265.c
@@ -224,6 +224,11 @@ static int libx265_encode_frame(AV
On Wed, Jan 21, 2015 at 04:52:33PM +, Derek Buitenhuis wrote:
> On 1/21/2015 4:46 PM, Timo Rothenpieler wrote:
> >> No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like
> >> libx264.c:
> >
> > Is falls back to that if pic->pict_type is NULL, but if pic is NULL, it
> > will just
On 1/21/2015 4:46 PM, Timo Rothenpieler wrote:
>> No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like libx264.c:
>
> Is falls back to that if pic->pict_type is NULL, but if pic is NULL, it
> will just crash.
I meant to send a reply saying I misread,
but responded to the wrong mail
> No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like libx264.c:
Is falls back to that if pic->pict_type is NULL, but if pic is NULL, it
will just crash.
signature.asc
Description: OpenPGP digital signature
___
ffmpeg-devel mailing lis
On 1/21/2015 4:28 PM, Michael Niedermayer wrote:
> this is missing pic==NULL checks or should be under the if() above
> or something like that
No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like libx264.c:
https://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavcodec/libx264.c;h=4da0e
On Wed, Jan 21, 2015 at 03:18:48PM +, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis
> ---
> libavcodec/libx265.c | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c
> index 923c750..5e4931a 100644
> --- a/libavcodec/libx265.c
Signed-off-by: Derek Buitenhuis
---
libavcodec/libx265.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c
index 923c750..5e4931a 100644
--- a/libavcodec/libx265.c
+++ b/libavcodec/libx265.c
@@ -226,6 +226,11 @@ static int libx265_encode_frame(AV