Page MenuHomeFreeBSD

iflib: Fix compiler warnings
ClosedPublic

Authored by stevek on Apr 3 2024, 8:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 15, 3:10 AM
Unknown Object (File)
Oct 2 2024, 6:46 PM
Unknown Object (File)
Oct 2 2024, 6:46 PM
Unknown Object (File)
Oct 2 2024, 6:46 PM
Unknown Object (File)
Oct 2 2024, 5:31 PM
Unknown Object (File)
Sep 30 2024, 9:19 PM
Unknown Object (File)
Sep 26 2024, 3:44 PM
Unknown Object (File)
Sep 26 2024, 5:22 AM

Details

Summary

Some of the QUAD sysctls are actually for unsigned quad values.
Switch to using UQUAD instead, as that is meant for unsigned.

Obtained from: Juniper Networks, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 56897
Build 53785: arc lint + arc unit

Event Timeline

stevek requested review of this revision.Apr 3 2024, 8:23 PM
jeffpc_josefsipek.net added inline comments.
sys/net/iflib.c
6854

Should the wrapped lines be indented one more char to maintain alignment?

LGTM. If the file was using normal style it wouldn't need the indentation fixup (which will add a lot of noise). I would maybe suggest instead to fix the indentation to not be unusual in lieu of the one char fixup.

This revision is now accepted and ready to land.Apr 12 2024, 8:07 PM
In D44620#1020403, @jhb wrote:

LGTM. If the file was using normal style it wouldn't need the indentation fixup (which will add a lot of noise). I would maybe suggest instead to fix the indentation to not be unusual in lieu of the one char fixup.

What about committing this change as-is and then doing a follow-up to correct the style as a separate review?

In D44620#1020403, @jhb wrote:

LGTM. If the file was using normal style it wouldn't need the indentation fixup (which will add a lot of noise). I would maybe suggest instead to fix the indentation to not be unusual in lieu of the one char fixup.

What about committing this change as-is and then doing a follow-up to correct the style as a separate review?

Yes, that's fine with me.

This revision was automatically updated to reflect the committed changes.