[vlc-devel] [PATCH 27/33] chromecast: better and less logs

Steve Lhomme robux4 at gmail.com
Wed Dec 23 21:35:33 CET 2015


On Wed, Dec 23, 2015 at 9:30 PM, Hugo Beauzée-Luyssen <hugo at beauzee.fr> wrote:
> On 12/23/2015 09:06 PM, Steve Lhomme wrote:
>>
>> On Wed, Dec 23, 2015 at 5:11 PM, Hugo Beauzée-Luyssen <hugo at beauzee.fr>
>> wrote:
>>>
>>> On 12/23/2015 12:58 PM, Steve Lhomme wrote:
>>>>
>>>>
>>>> ---
>>>>    modules/stream_out/chromecast/chromecast_ctrl.cpp | 22
>>>> +++++++++++++++-------
>>>>    1 file changed, 15 insertions(+), 7 deletions(-)
>>>>
>>>> diff --git a/modules/stream_out/chromecast/chromecast_ctrl.cpp
>>>> b/modules/stream_out/chromecast/chromecast_ctrl.cpp
>>>> index 1650efb..ae2a1a0 100644
>>>> --- a/modules/stream_out/chromecast/chromecast_ctrl.cpp
>>>> +++ b/modules/stream_out/chromecast/chromecast_ctrl.cpp
>>>> @@ -222,6 +222,10 @@ void intf_sys_t::processMessage(const
>>>> castchannel::CastMessage &msg)
>>>>    {
>>>>        const std::string & namespace_ = msg.namespace_();
>>>>
>>>> +#ifndef NDEBUG
>>>> +    msg_Dbg(p_stream,"processMessage: %s->%s %s", namespace_.c_str(),
>>>> msg.destination_id().c_str(), msg.payload_utf8().c_str());
>>>> +#endif
>>>> +
>>>>        if (namespace_ == NAMESPACE_DEVICEAUTH)
>>>>        {
>>>>            castchannel::DeviceAuthMessage authMessage;
>>>> @@ -259,7 +263,7 @@ void intf_sys_t::processMessage(const
>>>> castchannel::CastMessage &msg)
>>>>            }
>>>>            else
>>>>            {
>>>> -            msg_Err(p_stream, "Heartbeat command not supported: %s",
>>>> type.c_str());
>>>> +            msg_Warn(p_stream, "Heartbeat command not supported: %s",
>>>> type.c_str());
>>>>            }
>>>>
>>>>            json_value_free(p_data);
>>>> @@ -334,7 +338,7 @@ void intf_sys_t::processMessage(const
>>>> castchannel::CastMessage &msg)
>>>>            }
>>>>            else
>>>>            {
>>>> -            msg_Err(p_stream, "Receiver command not supported: %s",
>>>> +            msg_Warn(p_stream, "Receiver command not supported: %s",
>>>>                        msg.payload_utf8().c_str());
>>>>            }
>>>>
>>>> @@ -348,8 +352,9 @@ void intf_sys_t::processMessage(const
>>>> castchannel::CastMessage &msg)
>>>>            if (type == "MEDIA_STATUS")
>>>>            {
>>>>                json_value status = (*p_data)["status"];
>>>> -            msg_Dbg(p_stream, "Player state: %s",
>>>> -                    status[0]["playerState"].operator const char *());
>>>> +            msg_Dbg(p_stream, "Player state: %s sessionId:%d",
>>>> +                    status[0]["playerState"].operator const char *(),
>>>
>>>
>>>
>>> (const char*)status[0]["playerState"] looks more natural
>>>
>>>> +                    (int)(json_int_t) status[0]["mediaSessionId"]);
>>>>            }
>>>>            else if (type == "LOAD_FAILED")
>>>>            {
>>>> @@ -357,11 +362,14 @@ void intf_sys_t::processMessage(const
>>>> castchannel::CastMessage &msg)
>>>>                msgReceiverClose(appTransportId);
>>>>                vlc_mutex_lock(&lock);
>>>>                setConnectionStatus(CHROMECAST_CONNECTION_DEAD);
>>>> -            vlc_mutex_unlock(&lock);
>>>
>>>
>>>
>>> This is probably a leftover from a git rebase?
>>
>>
>> Yes, I saw it in a much further commit. JB fixed the push.
>>
>
> Unless I missed something, the pushed patch still removes the mutex unlock.

Oops, I misread. I'll send a fix with vlc_mutex_locker.

> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list