[FernetMenta]

+1 to Player.Process(xxx) to keep consistency.

Or even Player.xxx

The latter certainly not. We deal with info here only freaks are interested in and I don't want to mix this with the rest of Player or VideoPlayer labels.

[Gade]

+1 to Player.Process(xxx) to keep consistency.

Or even Player.xxx

The latter certainly not. We deal with info here only freaks are interested in and I don't want to mix this with the rest of Player or VideoPlayer labels.

Hehe, sounds fair enough

Whatever you guys decide will be fine.

[FernetMenta]

Here is the change: https://github.com/xbmc/xbmc/pull/10069

can someone verify that it works?

[sualfred]

I haven't set up a build environment. Could you trigger a test build?

[Hitcher]

Just triggered a build - should appear in the Windows test builds in about an hour.

[sualfred]

Thanks. Still have some issues. Let's continue on GitHub (https://github.com/xbmc/xbmc/pull/10069#...-230167781)

[KenV99]

At this point in time, have these new info label changes been propagated to python and JSON?

Using 17.0-ALPHA3 Git:20160721-a3f94c5 and executing this json code while a video file is successfully playing:

Code:

query = '{"jsonrpc": "2.0", "method": "XBMC.GetInfoLabels", "params": {"labels": ["Player.Process(VideoFps)"]}, "id": 1}'
result = xbmc.executeJSONRPC(query)
jsonr = loads(result)

returns a dict with an empty string result:

Code:

{u'jsonrpc': u'2.0', u'id': 1, u'result': {u'Player.Process(VideoFps)': u''}}

and

Code:

xbmc.getInfoLabel('Player.Process(VideoFps)')

echoes back the string: 'Player.Process(VideoFps)'

Am I using the wrong syntax?

[KenV99]

Ok, the problem was the capitalization detailed in @ronie 's post in the sticky.

No caps in the parenthesis (i.e. should be Player.Process(videofps).