cp: Address style issues from external review.
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F170538870: D50256.id.diff
	Sat, Sep 5, 7:40 AM

	F170406550: D50256.id155239.diff
	Fri, Sep 4, 4:35 PM

	Unknown Object (File)
	Wed, Sep 2, 11:02 AM

	Unknown Object (File)
	Mon, Aug 31, 11:12 AM

	Unknown Object (File)
	Sat, Aug 29, 11:13 AM

	Unknown Object (File)
	Thu, Aug 27, 11:01 AM

	Unknown Object (File)
	Tue, Aug 25, 10:25 AM

	Unknown Object (File)
	Mon, Aug 10, 6:53 AM

View All Files

Subscribers

imp

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 Not Applicable

Unit

Tests Not Applicable

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

54 lines

Diff 155239

View Options

cp: Address style issues from external review.ClosedPublicActions