Visitar URL original
Add IpHlpApi GetExtendedTcpTable and GetExtendedUdpTable by dbwiddis · Pull Request #1299 · java-native-access/jna · GitHub
Skip to content

Add IpHlpApi GetExtendedTcpTable and GetExtendedUdpTable - #1299

Merged
dbwiddis merged 1 commit into
java-native-access:masterfrom
dbwiddis:tcpudp
Jan 30, 2021
Merged

Add IpHlpApi GetExtendedTcpTable and GetExtendedUdpTable#1299
dbwiddis merged 1 commit into
java-native-access:masterfrom
dbwiddis:tcpudp

Conversation

@dbwiddis

@dbwiddis dbwiddis commented Jan 19, 2021

Copy link
Copy Markdown
Contributor

Also added some of the supporting structures sufficient to replicate most of netstat output for TCP and UDP.

matthiasblaesing
matthiasblaesing reviewed Jan 27, 2021
View reviewed changes
Comment thread contrib/platform/src/com/sun/jna/platform/win32/IPHlpAPI.java Resolved
Comment thread contrib/platform/src/com/sun/jna/platform/win32/IPHlpAPI.java Resolved
Comment thread contrib/platform/src/com/sun/jna/platform/win32/IPHlpAPI.java Resolved
Comment thread contrib/platform/src/com/sun/jna/platform/win32/IPHlpAPI.java Resolved
Comment thread contrib/platform/src/com/sun/jna/platform/win32/IPHlpAPI.java Resolved
@dbwiddis
dbwiddis force-pushed the tcpudp branch 2 times, most recently from 792f210 to 25013e1 Compare January 28, 2021 02:13
matthiasblaesing
matthiasblaesing reviewed Jan 28, 2021
View reviewed changes
Comment thread contrib/platform/src/com/sun/jna/platform/win32/IPHlpAPI.java Resolved
@dbwiddis
dbwiddis force-pushed the tcpudp branch 3 times, most recently from 4fdb129 to c9b50b8 Compare January 29, 2021 22:35
@dbwiddis

dbwiddis commented Jan 29, 2021

Copy link
Copy Markdown
Contributor Author

Test failures seem unrelated to this PR. I've seen that CrashProtectionTest failure more than a few times.

@matthiasblaesing

matthiasblaesing commented Jan 30, 2021

Copy link
Copy Markdown
Member

Change looks sane to me.

@dbwiddis
dbwiddis merged commit 4a5fe88 into java-native-access:master Jan 30, 2021
@dbwiddis
dbwiddis deleted the tcpudp branch January 30, 2021 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@matthiasblaesing matthiasblaesing matthiasblaesing left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dbwiddis @matthiasblaesing