Experimental Gotham addon, with additional functionality

[mvallevand]

Nothing has changed on that front. If its any quicker/slower, it'll most likely come down to Gotham changes. I haven't noticed any significant different myself though - with about 20 channels and a week of listings.

It seems like a combination of this that makes the difference

pCapabilities->bSupportsLastPlayedPosition = true;

Martin

[sub]

That shouldn't have any effect on loading the EPG. Might have an impact on loading the recordings though. Around about how many recordings did you have in your database?

[mvallevand]

I have 350 or so.

Martin

[sub]

I'll try an optimization tomorrow around the loading of playback position info.

[KRA]

EDL doesn't work on my build either. Playback through videos show edl data, but on pvr playback no edl info is found.

[sub]

EDL doesn't work on my build either. Playback through videos show edl data, but on pvr playback no edl info is found.

Yeah, unfortunately it seems to be a limitation of XBMC. Even though I've implemented support for EDL, they never call the GetRecordingEdl() function when you use http for streaming recordings. Grr.

[KRA]

Can I have source of new version to build ?

Never mind, didn't notice other thread

[mvallevand]

- resume position not dynamic with other clients

You never commented on this put I noticed that there may not be a /service?method=recording.watched.get. Do you plan on adding this?

Martin

[sub]

Do you mean in case another client played the file since the recording list was loaded?

There isn't currently an API for getting just this, but I could look to add one when I'm back. Thinking out loud - it's probably already doing the best thing for now, since the addon would have been incompatible with the base 3.1.1 release if I updated it to rely on an API that doesn't exist yet. For now it's just using APIs that already exist.

[mvallevand]

Do you mean in case another client played the file since the recording list was loaded?

There isn't currently an API for getting just this, but I could look to add one when I'm back. Thinking out loud - it's probably already doing the best thing for now, since the addon would have been incompatible with the base 3.1.1 release if I updated it to rely on an API that doesn't exist yet. For now it's just using APIs that already exist.

I'd like to see it in your code here to do a proper resume which gets called on Enter

Code:

int cPVRClientNextPVR::GetRecordingLastPlayedPosition(const PVR_RECORDING &recording)
{
  return recording.iLastPlayedPosition;
}

For compatibility that is what you could return if you don't get ok.

Martin