/gst-switch forked from duzy/gst-switch

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

Lots of cleanup and small fixes. #71

Merged
merged 9 commits into from Jan 7, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@mithro @landscape-bot @coveralls @hyades
@mithro
Member

mithro commented Dec 25, 2014

Fixes issues #45 and #52
@landscape-bot

This comment has been minimized.

Show comment
Hide comment
@landscape-bot

landscape-bot Dec 25, 2014

Code Health
The overall code health score did not change when comparing master 9cf4cf5 on mithro/gst-switch to master ff7910f on timvideos/gst-switch

108 new problems were found (including 78 errors and 12 code smells)
109 problems were fixed (including 79 errors and 12 code smells)

View the changes on Landscape.io

landscape-bot commented Dec 25, 2014

Code Health
The overall code health score did not change when comparing master 9cf4cf5 on mithro/gst-switch to master ff7910f on timvideos/gst-switch

108 new problems were found (including 78 errors and 12 code smells)
109 problems were fixed (including 79 errors and 12 code smells)

View the changes on Landscape.io
@landscape-bot

This comment has been minimized.

Show comment
Hide comment
@landscape-bot

landscape-bot Dec 29, 2014

Code Health
Code quality remained the same when pulling d5ef023 on mithro:gst-switch into 5d9212b on timvideos:gst-switch.

landscape-bot commented Dec 29, 2014

Code Health
Code quality remained the same when pulling d5ef023 on mithro:gst-switch into 5d9212b on timvideos:gst-switch.
@mithro

This comment has been minimized.

Show comment
Hide comment
@mithro

mithro Dec 29, 2014

Member

@hyades Any idea why these patches causes segfaults?
Member

mithro commented Dec 29, 2014

@hyades Any idea why these patches causes segfaults?
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 5, 2015

Coverage Status

Coverage increased (+3.71%) when pulling 8eecfef on mithro:master into 288a5eb on timvideos:master.

coveralls commented Jan 5, 2015

Coverage Status

Coverage increased (+3.71%) when pulling 8eecfef on mithro:master into 288a5eb on timvideos:master.
@landscape-bot

This comment has been minimized.

Show comment
Hide comment
@landscape-bot

landscape-bot Jan 5, 2015

Code Health
Code quality remained the same when pulling f3b7a84 on mithro:master into 288a5eb on timvideos:master.

landscape-bot commented Jan 5, 2015

Code Health
Code quality remained the same when pulling f3b7a84 on mithro:master into 288a5eb on timvideos:master.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 5, 2015

Coverage Status

Coverage increased (+3.71%) when pulling f3b7a84 on mithro:master into 288a5eb on timvideos:master.

coveralls commented Jan 5, 2015

Coverage Status

Coverage increased (+3.71%) when pulling f3b7a84 on mithro:master into 288a5eb on timvideos:master.
@landscape-bot

This comment has been minimized.

Show comment
Hide comment
@landscape-bot

landscape-bot Jan 7, 2015

Code Health
Code quality remained the same when pulling a7c328d on mithro:master into 5bb251d on timvideos:master.

landscape-bot commented Jan 7, 2015

Code Health
Code quality remained the same when pulling a7c328d on mithro:master into 5bb251d on timvideos:master.

hyades added a commit that referenced this pull request Jan 7, 2015

@hyades
Merge pull request #71 from mithro/master 
Lots of cleanup and small fixes.
19c5a6a

@hyades hyades merged commit 19c5a6a into timvideos:master Jan 7, 2015
1 check was pending

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment