Page MenuHomeFreeBSD
Differential D29814

bsd.prog.mk: Remove redundant check of the DEBUG_FLAGS
AbandonedPublic
Actions

Authored by 0mp on Apr 17 2021, 9:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 28, 10:25 PM
Unknown Object (File)
Fri, Jun 21, 10:46 PM
Unknown Object (File)
May 23 2024, 10:49 PM
Unknown Object (File)
May 13 2024, 3:26 AM
Unknown Object (File)
May 6 2024, 12:20 AM
Unknown Object (File)
May 1 2024, 10:38 AM
Unknown Object (File)
Apr 26 2024, 1:10 PM
Unknown Object (File)
Dec 24 2023, 7:25 PM
View All 29 Files
Subscribers
bdrewery
emaste
imp
yuripv

Details

Reviewers
jrtc27
emaste
arichardson
Group Reviewers
Contributor Reviews (src)
Summary

We already check if DEBUG_FLAGS is defined earlier in the same file, so
there is no need to check it again.

PR: 205036

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 38657
Build 35546: arc lint + arc unit

Event Timeline

0mp created this revision.Apr 17 2021, 9:05 PM
Herald added subscribers: emaste, imp, bdrewery. · View Herald TranscriptApr 17 2021, 9:05 PM
0mp requested review of this revision.Apr 17 2021, 9:05 PM
Harbormaster completed remote builds in B38657: Diff 87634.Apr 17 2021, 9:06 PM
yuripv added a subscriber: yuripv.Apr 17 2021, 10:41 PM
yuripv added inline comments.
share/mk/bsd.prog.mk
96

you seem to have lost one .endif

yuripv added inline comments.Apr 17 2021, 10:42 PM
share/mk/bsd.prog.mk
96

or no, you didn't, sorry.

0mp added a reviewer: Contributor Reviews (src).Apr 18 2021, 12:29 PM
0mp marked 2 inline comments as done.
0mp added reviewers: jrtc27, emaste, arichardson.Jul 8 2022, 7:41 PM

Change reviewers to the committers who last touched that file.

0mp abandoned this revision.Dec 13 2022, 2:34 PM

I don't think it is necessary to commit this style patch. The current version is readable. At the same time, the proposed change introduces a potentially confusing else statement, which has to be clarified with a comment.

Revision Contents
Changeset List

PathSize
share/
mk/
6 lines

Diff 87634

View Options

share/mk/bsd.prog.mk

Loading...