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)
Tue, Jul 28, 10:34 PM
Unknown Object (File)
Tue, Jul 28, 8:53 PM
Unknown Object (File)
Tue, Jul 28, 10:27 AM
Unknown Object (File)
Sun, Jul 26, 11:58 PM
Unknown Object (File)
Sun, Jul 26, 7:53 PM
Unknown Object (File)
Sat, Jul 25, 7:14 AM
Unknown Object (File)
Mon, Jul 20, 1:07 AM
Unknown Object (File)
Sun, Jul 19, 6:39 PM
View All Files
Subscribers
imp
patmaddox

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
patmaddox added a subscriber: 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...