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

Fix another instance of coverity #719001 #7355

Merged
merged 1 commit into from Jul 4, 2015

Conversation

Projects
None yet
3 participants
@tobbi
Contributor

tobbi commented Jun 28, 2015

So, I seemed to have missed that one last time.

@mkortstiege

This comment has been minimized.

Member

mkortstiege commented Jun 28, 2015

jenkins build this please

@MartijnKaijser

This comment has been minimized.

Member

MartijnKaijser commented Jun 28, 2015

please clarify the commit message what it fixes

@tobbi

This comment has been minimized.

Contributor

tobbi commented Jun 28, 2015

Fixed the commit message. Not sure if you need to re-trigger a build or not.

@MartijnKaijser

This comment has been minimized.

Member

MartijnKaijser commented Jun 28, 2015

build succeeded.

@MartijnKaijser

This comment has been minimized.

Member

MartijnKaijser commented Jul 3, 2015

jenkins build this please

@MartijnKaijser MartijnKaijser added this to the Isengard 16.0-alpha1 milestone Jul 3, 2015

MartijnKaijser added a commit that referenced this pull request Jul 4, 2015

Merge pull request #7355 from tobbi/coverity_719001
Fix another instance of coverity #719001

@MartijnKaijser MartijnKaijser merged commit dd97e81 into xbmc:master Jul 4, 2015

1 check passed

default Merged build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment