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.

Sign up for GitHub

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

fix: Don't use memcmp to compare IP_Ports. #2605

Merged
merged 1 commit into from
Jan 29, 2024

fix: Don't use `memcmp` to compare `IP_Port`s.

39aadf8
Select commit
Loading
Failed to load commit list.
Merged

fix: Don't use memcmp to compare IP_Ports. #2605

fix: Don't use `memcmp` to compare `IP_Port`s.
39aadf8
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 29, 2024 in 0s

67.50% of diff hit (target 73.72%)

View this Pull Request on Codecov

67.50% of diff hit (target 73.72%)

Annotations

Check warning on line 1449 in toxcore/network.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

toxcore/network.c#L1447-L1449 

Added lines #L1447 - L1449 were not covered by tests

Check warning on line 1451 in toxcore/network.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

toxcore/network.c#L1451 

Added line #L1451 was not covered by tests

Check warning on line 1463 in toxcore/network.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

toxcore/network.c#L1462-L1463 

Added lines #L1462 - L1463 were not covered by tests

Check warning on line 1473 in toxcore/network.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

toxcore/network.c#L1468-L1473 

Added lines #L1468 - L1473 were not covered by tests

Check warning on line 1476 in toxcore/network.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

toxcore/network.c#L1476 

Added line #L1476 was not covered by tests
View more details on Codecov