diff mbox

[FFmpeg-devel,1/1] make headers compile in c++ mode

Message ID 20180426124736.49904-2-daniel@danieloberhoff.de
State New
Headers show

Commit Message

Daniel Oberhoff April 26, 2018, 12:47 p.m. UTC 
---
 include/ffnvcodec/dynlink_loader.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Nicolas George April 26, 2018, 12:49 p.m. UTC | #1 
Daniel Oberhoff (2018-04-26):
> ---
>  include/ffnvcodec/dynlink_loader.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

I may be missing something obvious, but I am not seeing this file in our
repository.

Regards,
Daniel Oberhoff April 26, 2018, 12:51 p.m. UTC | #2 
> On 26. Apr 2018, at 14:49, Nicolas George <george@nsup.org> wrote:
> 
> Daniel Oberhoff (2018-04-26):
>> ---
>> include/ffnvcodec/dynlink_loader.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> I may be missing something obvious, but I am not seeing this file in our
> repository.

sorry, its from the nv-codec-headers repo. Should have stated that in the subject. Is this not the right list for patches there?

Daniel
wm4 April 26, 2018, 1:03 p.m. UTC | #3 
On Thu, 26 Apr 2018 14:51:09 +0200
Daniel Oberhoff <danieloberhoff@googlemail.com> wrote:

> > On 26. Apr 2018, at 14:49, Nicolas George <george@nsup.org> wrote:
> > 
> > Daniel Oberhoff (2018-04-26):  
> >> ---
> >> include/ffnvcodec/dynlink_loader.h | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)  
> > 
> > I may be missing something obvious, but I am not seeing this file in our
> > repository.  
> 
> sorry, its from the nv-codec-headers repo. Should have stated that in the subject. Is this not the right list for patches there?

The repository is a ffmpeg project, so this is the right place.
Timo Rothenpieler April 26, 2018, 1:46 p.m. UTC | #4 
On 26.04.2018 14:47, Daniel Oberhoff wrote:
> ---
>  include/ffnvcodec/dynlink_loader.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/ffnvcodec/dynlink_loader.h b/include/ffnvcodec/dynlink_loader.h
> index 352a0c8..3b0a284 100644
> --- a/include/ffnvcodec/dynlink_loader.h
> +++ b/include/ffnvcodec/dynlink_loader.h
> @@ -114,7 +114,7 @@
>                                               \
>      n##_free_functions(functions);           \
>                                               \
> -    f = *functions = calloc(1, sizeof(*f));  \
> +    f = *functions = (T*)calloc(1, sizeof(*f));  \
>      if (!f)                                  \
>          return -1;                           \
>                                               \

applied
diff mbox

Patch

diff --git a/include/ffnvcodec/dynlink_loader.h b/include/ffnvcodec/dynlink_loader.h
index 352a0c8..3b0a284 100644
--- a/include/ffnvcodec/dynlink_loader.h
+++ b/include/ffnvcodec/dynlink_loader.h
@@ -114,7 +114,7 @@ 
                                              \
     n##_free_functions(functions);           \
                                              \
-    f = *functions = calloc(1, sizeof(*f));  \
+    f = *functions = (T*)calloc(1, sizeof(*f));  \
     if (!f)                                  \
         return -1;                           \
                                              \