Page MenuHomeFreeBSD

Fix a comment post-r320347
ClosedPublic

Authored by jhibbits on Jun 26 2017, 3:02 AM.
Tags
None
Referenced Files
F107823804: D11364.diff
Sat, Jan 18, 1:00 PM
Unknown Object (File)
Dec 12 2024, 1:28 PM
Unknown Object (File)
Oct 15 2024, 4:08 AM
Unknown Object (File)
Oct 10 2024, 3:53 AM
Unknown Object (File)
Sep 27 2024, 11:40 PM
Unknown Object (File)
Sep 25 2024, 3:14 AM
Unknown Object (File)
Sep 22 2024, 7:01 AM
Unknown Object (File)
Sep 21 2024, 5:13 PM
Subscribers

Details

Summary

As of r320347 i386 is the only arch with 32-bit time_t. Instead of
just fixing the time32_t comment, also simplify the condition.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib added inline comments.
sys/compat/freebsd32/freebsd32.h
48 ↗(On Diff #30078)
#ifdef __amd64__
typedef int32_t time32_t;
#else
...
This revision is now accepted and ready to land.Jun 26 2017, 11:39 AM

Perhaps apply the same treatment to sys/compat/freebsd32/freebsd32_misc.c? (and sys/net/bpf.c)?

jhibbits edited edge metadata.

Add more files that only need amd64 guards.

This revision now requires review to proceed.Jun 26 2017, 2:49 PM
This revision is now accepted and ready to land.Jun 26 2017, 3:29 PM

I was going to suggest the same change.

This revision was automatically updated to reflect the committed changes.