Page MenuHomeFreeBSD
Differential D38172

w: Complete libxo transition.
ClosedPublic
Actions

Authored by des on Jan 23 2023, 4:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 8, 7:54 PM
Unknown Object (File)
Thu, Mar 6, 6:54 PM
Unknown Object (File)
Wed, Feb 26, 11:46 PM
Unknown Object (File)
Tue, Feb 25, 7:48 AM
Unknown Object (File)
Tue, Feb 25, 6:52 AM
Unknown Object (File)
Mon, Feb 24, 5:37 AM
Unknown Object (File)
Mon, Feb 24, 1:38 AM
Unknown Object (File)
Sun, Feb 23, 12:04 PM
View All 62 Files
Subscribers
imp
markj

Details

Reviewers
markj
Group Reviewers
Klara
Commits
rGa6edc5b39209: w: Complete libxo transition.
rGd90ff31ae5cd: w: Complete libxo transition.
Summary

MFC after: 1 week
Sponsred by: Klara, Inc.

Diff Detail

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

Event Timeline

des created this revision.Jan 23 2023, 4:24 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 23 2023, 4:24 PM
des requested review of this revision.Jan 23 2023, 4:24 PM
Harbormaster completed remote builds in B49197: Diff 115452.Jan 23 2023, 4:24 PM
markj added a subscriber: markj.Jan 23 2023, 7:36 PM
markj added inline comments.
usr.bin/w/w.c
475–476

It looks like xo_finish() returns a number of bytes written, so shouldn't we be checking for a result < 0?

des updated this revision to Diff 115510.Jan 24 2023, 8:49 AM

Fix xo_finish() check

Harbormaster completed remote builds in B49204: Diff 115510.Jan 24 2023, 8:49 AM
des marked an inline comment as done.Jan 24 2023, 8:49 AM
des added inline comments.
usr.bin/w/w.c
475–476

You are absolutely right.

markj accepted this revision as: markj.Jan 24 2023, 2:06 PM
This revision is now accepted and ready to land.Jan 24 2023, 2:06 PM
Closed by commit rGd90ff31ae5cd: w: Complete libxo transition. (authored by des). · Explain WhyJan 25 2023, 6:23 AM
This revision was automatically updated to reflect the committed changes.
des marked an inline comment as done.
des added a commit: rGd90ff31ae5cd: w: Complete libxo transition..
des added a commit: rGa6edc5b39209: w: Complete libxo transition..Feb 9 2023, 8:41 PM

Revision Contents
Changeset List

PathSize
usr.bin/
w/
17 lines

Diff 115708

View Options

usr.bin/w/w.c

Loading...