Skip to content

Updates to Swift UDP#42

Merged
agnosticdev merged 5 commits into
mainfrom
agnosticdev/UpdateUDP
Jul 6, 2026
Merged

Updates to Swift UDP#42
agnosticdev merged 5 commits into
mainfrom
agnosticdev/UpdateUDP

Conversation

@agnosticdev

@agnosticdev agnosticdev commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Stubbed out logic for checksum offload functionality and added unit tests.
Added DataTransferSnapshot metric support and added unit tests.

@agnosticdev agnosticdev requested review from rnro and rpaulo July 3, 2026 01:13
Comment thread Sources/SwiftNetwork/Protocols/UDPProtocol.swift Outdated
Comment thread Sources/SwiftNetwork/Protocols/Checksum.swift Outdated
Comment thread Sources/SwiftNetwork/Protocols/Checksum.swift Outdated
Comment thread Sources/SwiftNetwork/Protocols/UDPProtocol.swift Outdated

@tfpauly tfpauly left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine modulo style fixes

Comment thread Sources/SwiftNetwork/Protocols/Checksum.swift Outdated
Comment thread Sources/SwiftNetwork/Protocols/UDPProtocol.swift Outdated

@rpaulo rpaulo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only style comments left

@agnosticdev agnosticdev merged commit fe09a97 into main Jul 6, 2026
23 checks passed
@agnosticdev agnosticdev deleted the agnosticdev/UpdateUDP branch July 6, 2026 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants