FSSDawid/xbmc — comment on commit 2982626186b60c822a350033e55619058107350b
File: xbmc/interfaces/json-rpc/PVROperations.cpp line 117
By FSSDawid:

Yes that was exactly how I was doing it but It will not compile this
way :/ I will ask a c++ guy

Sent from my iPhone

On 11 gru 2011, at 16:20, Lars Op den Kamp

wrote:

> nope, you need to check for the return value of GetCurrentChannel().
> this should be
> '''C++
> CPVRChannel currentChannel;
> if (g_PVRManager.GetCurrentChannel(currentChannel))
> {
> // do stuff
> }
> '''
>
> if you don't check the return value, you're just using the default values from CPVRChannel's constructor.
>
> I also noticed that I'm not checking whether the manager is running in GetCurrentChannel(). I'll fix that.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/FSSDawid/xbmc/commit/2982626186b60c822a350033e55619058107350b#commitcomment-785068