Page MenuHomeFreeBSD
Differential D51876

last: Mild style cleanup
ClosedPublic
Actions

Authored by des on Tue, Aug 12, 3:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Aug 29, 12:27 AM
Unknown Object (File)
Mon, Aug 25, 10:03 AM
Unknown Object (File)
Mon, Aug 25, 5:33 AM
Unknown Object (File)
Sun, Aug 24, 11:41 AM
Unknown Object (File)
Fri, Aug 22, 3:44 PM
Unknown Object (File)
Fri, Aug 22, 3:32 PM
Unknown Object (File)
Thu, Aug 21, 6:49 AM
Unknown Object (File)
Tue, Aug 19, 11:07 PM
View All 12 Files
Subscribers
imp
markj

Details

Reviewers
markj
Group Reviewers
Klara
Commits
rG40c2b622d825: last: Mild style cleanup
Summary
  • Prefer explicit NULL comparisons to using pointers as booleans
  • Prefer explicit 0 comparisons to using strcmp() as a predicate
  • Avoid use of strcpy()

Sponsored by: Klara, Inc.

Diff Detail

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

Event Timeline

des created this revision.Tue, Aug 12, 3:56 PM
Herald added a subscriber: imp. · View Herald TranscriptTue, Aug 12, 3:56 PM
des requested review of this revision.Tue, Aug 12, 3:56 PM
Harbormaster completed remote builds in B66198: Diff 160250.Tue, Aug 12, 3:56 PM
markj accepted this revision as: markj.Tue, Aug 12, 4:37 PM
markj added a subscriber: markj.
markj added inline comments.
usr.bin/last/last.c
513

Not a problem in practice, but it's not very nice that this helper function sometimes returns a malloc()ed buffer, and sometimes not.

This revision is now accepted and ready to land.Tue, Aug 12, 4:37 PM
Closed by commit rG40c2b622d825: last: Mild style cleanup (authored by des). · Explain WhyTue, Aug 12, 5:21 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG40c2b622d825: last: Mild style cleanup.

Revision Contents
Changeset List

PathSize
usr.bin/
last/
27 lines

Diff 160250

View Options

usr.bin/last/last.c

Loading...