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

sys/shell: fixed percentage calc of ping6 shell cmd #3278

Merged
merged 1 commit into from Jun 30, 2015

Conversation

haukepetersen
Copy link
Contributor

I was wondering why I always got 0% packet loss, even when losing one out of 100 -> 1%. Turns out the percentage calculation was wrong...

@haukepetersen haukepetersen added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer NSTF labels Jun 30, 2015
@haukepetersen haukepetersen added this to the Release 2015.06 milestone Jun 30, 2015
@miri64
Copy link
Member

miri64 commented Jun 30, 2015

ACK and go, when travis is happy.

@miri64 miri64 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jun 30, 2015
OlegHahm added a commit that referenced this pull request Jun 30, 2015
sys/shell: fixed percentage calc of ping6 shell cmd
@OlegHahm OlegHahm merged commit 3651f5f into RIOT-OS:master Jun 30, 2015
@haukepetersen haukepetersen deleted the fix_sc_pin_percentage branch June 30, 2015 14:40
@miri64 miri64 added the Area: network Area: Networking label Sep 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants