Page MenuHomeFreeBSD
Differential D434

Remove the LL abomination
ClosedPublic
Actions

Authored by davide on Jul 17 2014, 11:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 28, 5:06 PM
Unknown Object (File)
Jan 31 2024, 2:17 AM
Unknown Object (File)
Dec 22 2023, 8:25 PM
Unknown Object (File)
Nov 17 2023, 6:11 AM
Unknown Object (File)
Aug 24 2023, 10:24 AM
Unknown Object (File)
Jul 3 2023, 12:25 PM
Unknown Object (File)
May 10 2023, 7:35 PM
Unknown Object (File)
Mar 6 2023, 3:53 AM
View All 45 Files
Subscribers
None

Details

Reviewers
davide
mav
Summary

Requested by Bruce Evans

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

davide updated this revision to Diff 753.Jul 17 2014, 11:20 PM
davide retitled this revision from to Remove the LL abomination.
davide updated this object.
davide edited the test plan for this revision. (Show Details)
davide added a reviewer: mav.Jul 18 2014, 8:19 PM
mav edited edge metadata.Jul 19 2014, 12:48 PM

Not sure I am the best reviewer for this style question. My present thought is that it would possibly look better mapped to INT64_MAX.

davide added a comment.Aug 20 2014, 3:38 PM

Bruce Evans reviewed this privately.
Let's copy his comment here:

"OK. It has some subtleties and tiny unportabilities which are too arcane
to mention in comments or commit messages.

Please plan to remove the other 2 LL's in the file for the same reason.
Might be worth explaining in the commit message then."

davide accepted this revision.Aug 20 2014, 3:52 PM
davide added a reviewer: davide.

bde@ accepted this revision privately.

This revision is now accepted and ready to land.Aug 20 2014, 3:52 PM
davide closed this revision.Aug 20 2014, 5:31 PM

Committed (r270227)

Revision Contents
Changeset List

PathSize
sys/
sys/
2 lines

Diff 753

View Options

sys/sys/time.h

Loading...