Page MenuHomeFreeBSD
Differential D41414

lastlogin: Complete libxo transition.
ClosedPublic
Actions

Authored by yanhaowang on Aug 10 2023, 10:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 24, 9:00 PM
Unknown Object (File)
Sun, Nov 16, 8:16 AM
Unknown Object (File)
Sun, Nov 16, 7:08 AM
Unknown Object (File)
Sun, Nov 16, 4:58 AM
Unknown Object (File)
Sun, Nov 16, 2:11 AM
Unknown Object (File)
Sun, Nov 16, 2:07 AM
Unknown Object (File)
Sun, Nov 16, 1:58 AM
Unknown Object (File)
Sat, Nov 15, 9:33 PM
View All Files
Subscribers
guest-patmaddox
imp

Details

Reviewers
lwhsu
des
Commits
rG7d5fd25ab8f5: lastlogin(8): Complete libxo transition

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

yanhaowang created this revision.Aug 10 2023, 10:57 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 10 2023, 10:57 PM
yanhaowang requested review of this revision.Aug 10 2023, 10:57 PM
Harbormaster completed remote builds in B53039: Diff 125828.Aug 10 2023, 10:57 PM
yanhaowang added reviewers: lwhsu, des.Aug 10 2023, 10:58 PM
yanhaowang updated this revision to Diff 125902.Aug 12 2023, 2:19 AM
  • Remove unnecessary space line.
Harbormaster completed remote builds in B53065: Diff 125902.Aug 12 2023, 2:19 AM
des requested changes to this revision.Aug 17 2023, 3:09 PM
des added inline comments.
usr.sbin/lastlogin/lastlogin.c
150

The indentation here seems wrong.

173

There is no need to call xo_finish() here, since we haven't emitted anything.

This revision now requires changes to proceed.Aug 17 2023, 3:09 PM
yanhaowang updated this revision to Diff 126293.Aug 21 2023, 12:30 PM
  • Remove unnecessary xo_finish function.
  • Use number style.
Harbormaster completed remote builds in B53199: Diff 126293.Aug 21 2023, 12:30 PM
yanhaowang marked 2 inline comments as done.Aug 21 2023, 12:31 PM
des accepted this revision.Aug 26 2023, 11:49 PM
This revision is now accepted and ready to land.Aug 26 2023, 11:49 PM
guest-patmaddox added a subscriber: guest-patmaddox.Oct 19 2023, 9:46 PM
Closed by commit rG7d5fd25ab8f5: lastlogin(8): Complete libxo transition (authored by yanhaowang, committed by lwhsu). · Explain WhyOct 12 2024, 6:30 PM
This revision was automatically updated to reflect the committed changes.
lwhsu added a commit: rG7d5fd25ab8f5: lastlogin(8): Complete libxo transition.

Revision Contents
Changeset List

PathSize
usr.sbin/
lastlogin/
5 lines

Diff 144737

View Options

usr.sbin/lastlogin/lastlogin.c

Loading...