dee

Merge lp://qastaging/~charlesk/dee/fix-904995 into lp://qastaging/dee

  1. fix-904995
  2. Merge into trunk
Proposed by Charles Kerr
Status: Needs review
Proposed branch: lp://qastaging/~charlesk/dee/fix-904995
Merge into: lp://qastaging/dee
Diff against target: 45 lines (+22/-13)
1 file modified
tests/test-benchmark.c (+22/-13)
To merge this branch: bzr merge lp://qastaging/~charlesk/dee/fix-904995
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Francis Ginther Pending
Review via email: mp+210897@code.qastaging.launchpad.net

Commit message

Fix a potential divide-by-zero error detected by Coverity.

Description of the change

Fix (or, rather, silence) a potential divide-by-zero error detected by Coverity.

This patch fell behind the couch cushions awhile back, but I was cleaning out my branches and found it... good thing it's not an important bug.

To post a comment you must log in.
lp://qastaging/~charlesk/dee/fix-904995 updated
345. By Charles Kerr

sync with lp:dee

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:344
http://jenkins.qa.ubuntu.com/job/dee-ci/16/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-trusty-amd64-ci/4
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-trusty-armhf-ci/4
        deb: http://jenkins.qa.ubuntu.com/job/dee-trusty-armhf-ci/4/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-trusty-i386-ci/4

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/dee-ci/16/rebuild

review: Approve (continuous-integration)
Reply
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:345
http://jenkins.qa.ubuntu.com/job/dee-ci/17/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-trusty-amd64-ci/5
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-trusty-armhf-ci/5
        deb: http://jenkins.qa.ubuntu.com/job/dee-trusty-armhf-ci/5/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-trusty-i386-ci/5

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/dee-ci/17/rebuild

review: Approve (continuous-integration)
Reply

Unmerged revisions

345. By Charles Kerr

sync with lp:dee

344. By Charles Kerr

silence an improbable div-by-zero coverity warning (bug #904995)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: