Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Pydocs] Add info for Player and RenderCapture #4789

Merged
merged 1 commit into from May 25, 2014
Merged

[Pydocs] Add info for Player and RenderCapture #4789

merged 1 commit into from May 25, 2014

Conversation

KenV99
Copy link
Contributor

@KenV99 KenV99 commented May 24, 2014

  1. Added information in comments to be pulled into PyDocs describing the
    shortcomings when calling xbmc.player.getPlayingFile() for LiveTV.
    Specifically that pvr:// url's are not translated into usable filenames
    or streams.
  2. Added information to xbmc.CapturePlayer regarding the functions
    getHeight() and getWidth() which return 0 prior to calling
    CapturePlayer.capture(). Also added that getAspectRatio() may be called
    prior to issuing a capture() request.

1) Added information in comments to be pulled into PyDocs describing the
shortcomings when calling xbmc.player.getPlayingFile() for LiveTV.
Specifically that pvr:// url's are not translated into usable filenames
or streams.
2) Added information to xbmc.CapturePlayer regarding the functions
getHeight() and getWidth() which return 0 prior to calling
CapturePlayer.capture(). Also added that getAspectRatio() may be called
prior to issuing a capture() request.
@MartijnKaijser
Copy link
Member

thank you :)
(for reference. never work in master branch. always make separate branch. doing a frorce push to your branch will auto update the pull request. so no need to create a new one)
jenkins build this please

@KenV99
Copy link
Contributor Author

KenV99 commented May 24, 2014

Ok. My past use of github was always with a small number of ppl. I'm still learning how to negotiate with the more advanced stuff from the Git Bash shell. Again, thanks for the patience.

@jmarshallnz
Copy link
Contributor

No worries - we appreciate your contribution (and taking the time to learn new stuff!)

jmarshallnz added a commit that referenced this pull request May 25, 2014
[Pydocs] Add info for Player and RenderCapture
@jmarshallnz jmarshallnz merged commit e4b2c4a into xbmc:master May 25, 2014
@MartijnKaijser MartijnKaijser added this to the Helix 14.0-alpha1 milestone May 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants