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

Rma fixes #7

Merged
merged 2 commits into from
May 7, 2014
Merged

Rma fixes #7

merged 2 commits into from
May 7, 2014

Conversation

busterb
Copy link

@busterb busterb commented May 7, 2014

The first commit is just code formatting updates to better match other sigar conventions.

The second commit fixes a negative array index access bug.

Brent Cook added 2 commits May 7, 2014 05:53
When in Rome, code as the Romans do.
move wraparound handlers to helper functions
@thirty8super
Copy link

Whoops thanks!

Sent from my iPad

On May 7, 2014, at 6:21 AM, busterb notifications@github.com wrote:

The first commit is just code formatting updates to better match other sigar conventions.

The second commit fixes a negative array index access bug.

You can merge this Pull Request by running

git pull https://github.com/boundary/sigar rma_fixes
Or view, comment on, or merge it at:

#7

Commit Summary

adjust code format to fit the rest of the project
do not reference a negative array offset when current_pos = 0
File Changes

M include/sigar.h (39)
M src/sigar_rma.c (108)
Patch Links:

https://github.com/boundary/sigar/pull/7.patch
https://github.com/boundary/sigar/pull/7.diff

Reply to this email directly or view it on GitHub.

thirty8super added a commit that referenced this pull request May 7, 2014
@thirty8super thirty8super merged commit c8070a6 into master May 7, 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
2 participants