GitHub user Katana-Steel opened a pull request:
https://github.com/apache/libcloud/pull/799
refactored _parse_arp_table
## execution of the suggested refactor
### Description
passing each command output to their own parsing function
### Status
- done, ready for review
### Checklist (tick everything that applies)
- [X] [Code
linting](http://libcloud.readthedocs.org/en/latest/development.html#code-style-guide)
(required, can be done after the PR checks)
- [X] Documentation
- [ ] [Tests](http://libcloud.readthedocs.org/en/latest/testing.html)
- [ ]
[ICLA](http://libcloud.readthedocs.org/en/latest/development.html#contributing-bigger-changes)
(required for bigger changes)
split _parse_arp_table into 1 general and 2 specialized functions
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Katana-Steel/libcloud
libvirt/refactor_mac2ip_addr_parsing
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/libcloud/pull/799.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #799
----
commit aed44bc54c30b0adbded9976f8b9ce3ec0b735c8
Author: Rene Kjellerup <[email protected]>
Date: 2016-06-04T00:02:19Z
refactored _parse_arp_table
split _parse_arp_table into 1 general and 2 specialized functions
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---