cp: Address style issues from external review.
ClosedPublic
Actions

Authored by des on May 8 2025, 6:03 PM.

Details

Reviewers

markj
allanjude

Group Reviewers

Klara

Commits

rGa4f6317bb64b: cp: Address style issues from external review.

Summary

Correctly indent some comment blocks.
Avoid using strcpy(), even when we know it's safe.
Drop the (void) casts from fts_set() calls which can never fail.

Fixes: 82fc0d09e862
Sponsored by: Klara, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 64009
Build 60893: arc lint + arc unit

Event Timeline

des created this revision.May 8 2025, 6:03 PM

Herald added a subscriber: imp. · View Herald TranscriptMay 8 2025, 6:03 PM

des requested review of this revision.May 8 2025, 6:03 PM

Harbormaster completed remote builds in B64009: Diff 155149.May 8 2025, 6:03 PM

des added a child revision: D50257: cp: Fix dead link case..May 8 2025, 6:03 PM

markj accepted this revision as: markj.May 8 2025, 11:28 PM

This revision is now accepted and ready to land.May 8 2025, 11:28 PM

allanjude accepted this revision.May 9 2025, 12:51 PM

Closed by commit rGa4f6317bb64b: cp: Address style issues from external review. (authored by des). · Explain WhyMay 10 2025, 8:58 AM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGa4f6317bb64b: cp: Address style issues from external review..

Revision Contents
Changeset List

Path

Size

bin/

cp/

cp.c

60 lines

Diff 155149

View Options

cp: Address style issues from external review.ClosedPublicActions