lua-geoip
lua-geoip copied to clipboard
Commit of working asnum.c, to perform ASN lookups
This has been tested empirically and found to work for name, addr, and ipnum lookups. Still working on the actual test cases; would appreciate any feedback.
Thank you for your contribution! Please provide basic autotests and add yourself to the AUTHORS file so I can merge this.
On 15 дек. 2015 г., at 22:09, Robison Jacka [email protected] wrote:
This has been tested empirically and found to work for name, addr, and ipnum lookups. Still working on the actual test cases; would appreciate any feedback.
You can view, comment on, or merge this pull request online at:
https://github.com/agladysh/lua-geoip/pull/13
Commit Summary
Commit of working asnum.c, to perform ASN lookups File Changes
A src/asnum.c (228) Patch Links:
https://github.com/agladysh/lua-geoip/pull/13.patch https://github.com/agladysh/lua-geoip/pull/13.diff — Reply to this email directly or view it on GitHub.
This should be finished now. Found a dumb memory leak on my part, and finally squashed it. Also finished the tests & profiling for geodb.asnum. Please take a look, and let me know if you see anything that needs updating/fixing. Thanks!
Thank you for your pull request. Please see my questions.
@robison ping?