diff mbox series

[FFmpeg-devel] libavutil/timer: Fix clang reserved-user-defined-literal

Message ID 20210313042037.2932884-1-ccom@randomderp.com
State Accepted
Headers show
Series [FFmpeg-devel] libavutil/timer: Fix clang reserved-user-defined-literal | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Christopher Degawa March 13, 2021, 4:20 a.m. UTC
clang errors when compiling with C++11 about needing spaces between
literal and identifier

Signed-off-by: Christopher Degawa <ccom@randomderp.com>
---
 libavutil/timer.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Anton Khirnov March 13, 2021, 6:56 a.m. UTC | #1
Quoting Christopher Degawa (2021-03-13 05:20:37)
> clang errors when compiling with C++11 about needing spaces between
> literal and identifier

Why would you compile it with C++11? It's a private header, is it not?
Christopher Degawa March 13, 2021, 8:11 a.m. UTC | #2
On Sat, Mar 13, 2021 at 01:09 Chris Degawa <cddegawa@gmail.com> wrote:

>
>
> > On Mar 13, 2021, at 00:57, Anton Khirnov <anton@khirnov.net> wrote:
> >
> > Quoting Christopher Degawa (2021-03-13 05:20:37)
> >> clang errors when compiling with C++11 about needing spaces between
> >> literal and identifier
> >
> > Why would you compile it with C++11? It's a private header, is it not?


The error occurred when compiling decklink_dec.cpp
>
Since I realized my email did not send out based on looking at the ml
archive, resending using the right email
James Almer March 13, 2021, 2:44 p.m. UTC | #3
On 3/13/2021 5:11 AM, Christopher Degawa wrote:
> On Sat, Mar 13, 2021 at 01:09 Chris Degawa <cddegawa@gmail.com> wrote:
> 
>>
>>
>>> On Mar 13, 2021, at 00:57, Anton Khirnov <anton@khirnov.net> wrote:
>>>
>>> Quoting Christopher Degawa (2021-03-13 05:20:37)
>>>> clang errors when compiling with C++11 about needing spaces between
>>>> literal and identifier
>>>
>>> Why would you compile it with C++11? It's a private header, is it not?
> 
> 
> The error occurred when compiling decklink_dec.cpp

Yeah, starting with 9e2e6f935bed329a7c2eabae7de02cccd88e2f26 
decklink_dec.cpp includes libavutil/internal.h

Will apply, thanks.
diff mbox series

Patch

diff --git a/libavutil/timer.h b/libavutil/timer.h
index 0bb353cfce..737bfc160e 100644
--- a/libavutil/timer.h
+++ b/libavutil/timer.h
@@ -87,7 +87,7 @@ 
         if (((tcount + tskip_count) & (tcount + tskip_count - 1)) == 0) { \
             int i;                                                        \
             av_log(NULL, AV_LOG_ERROR,                                    \
-                   "%7"PRIu64" " FF_TIMER_UNITS " in %s,%8d runs,%7d skips",          \
+                   "%7" PRIu64 " " FF_TIMER_UNITS " in %s,%8d runs,%7d skips",          \
                    tsum * 10 / tcount, id, tcount, tskip_count);          \
             for (i = 0; i < 32; i++)                                      \
                 av_log(NULL, AV_LOG_VERBOSE, " %2d", av_log2(2*thistogram[i]));\