Page MenuHomeFreeBSD

posixshm_test.c: tautology in test
ClosedPublic

Authored by rlibby on Dec 10 2020, 10:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jul 28, 1:14 AM
Unknown Object (File)
Sat, Jul 12, 10:06 PM
Unknown Object (File)
Jun 27 2025, 3:26 AM
Unknown Object (File)
Jun 26 2025, 1:49 AM
Unknown Object (File)
Jun 25 2025, 1:06 PM
Unknown Object (File)
Jun 25 2025, 8:31 AM
Unknown Object (File)
Jun 24 2025, 12:43 AM
Unknown Object (File)
Jun 21 2025, 5:32 AM
Subscribers

Details

Summary

gcc -Wtype-limits complained about unsigned >= 0. Cast to signed so
that it might check for underflow.


Alternately, we might just delete the checks.

Test Plan
env MAKEOBJDIRPREFIX=/usr/obj/gcc6 CROSS_TOOLCHAIN=amd64-gcc make buildworld
env MAKEOBJDIRPREFIX=/usr/obj/clang make buildworld

... I'll look into actually running this test too.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 35331
Build 32260: arc lint + arc unit

Event Timeline

The in-kernel source for the sysctl is also unsigned.
I think that the checks can be safely dropped.

In D27564#615982, @kib wrote:

The in-kernel source for the sysctl is also unsigned.
I think that the checks can be safely dropped.

I agree. I can't remember why I added the checks.

feedback: just remove the checks

This revision is now accepted and ready to land.Dec 11 2020, 12:43 AM
This revision was automatically updated to reflect the committed changes.