Re: [PATCH] Fix building with Clang

On 12/20/2021 10:47 AM, Reimar Döffinger wrote:

>
>> On 10 Dec 2021, at 07:48, Brad Smith <[email protected]> wrote:
>>
>> Fix building with Clang.
>>
>>
>> ./stream/stream.h:322:46: error: invalid suffix on literal; C++11 requires a space between literal and identifier [-Wreserved-user-defined-literal]
>>   mp_dbg(MSGT_DEMUX, MSGL_DBG3, "seek to 0x%"PRIX64"\n", pos);
>>                                              ^
>>
>> ./stream/stream.h:326:49: error: invalid suffix on literal; C++11 requires a space between literal and identifier [-Wreserved-user-defined-literal]
>>            "Invalid seek to negative position %"PRIx64"!\n", pos);
> Sorry for the late reply, but I think this needs a bit more details/explanation.
> 1) I’ve quickly tested on macOS which uses clang and it has no issues, so this can’t be a problem with clang in general I think?
> 2) MPlayer is not C++11, in fact it is not C++ at all, so this messages does not really make sense. Sure it’s not a bug in clang, or maybe you used clang++ instead of clang as compiler?
>
> Now the change is not a big deal, it’s just that I am not a big fan of working around issues that are not properly understood, also because if it’s a real issue the commit message should make it clear how we can avoid re-introducing such issues in the future.

Hi Reimar,

I'll provide some further context to the error messages when I get home 
later on today.

_______________________________________________
MPlayer-dev-eng mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng