glabel(8): Fix debug message for already existing labels
ClosedPublic
Actions

Authored by michaelo on Wed, Apr 15, 8:13 AM.

Details

Reviewers

mav
markj
imp

Commits

rG92c69c791c43: glabel(8): Fix debug message for already existing labels

Summary

Remove the duplicate (incorrect) name.

PR: 294542
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 72232
Build 69115: arc lint + arc unit

Event Timeline

michaelo requested review of this revision.Wed, Apr 15, 8:13 AM

michaelo created this revision.

Harbormaster completed remote builds in B72212: Diff 175536.Wed, Apr 15, 8:13 AM

markj added inline comments.Wed, Apr 15, 1:23 PM

sys/geom/label/g_label.c
255	Just don't print `pp2->name` at all? We already know it's the same as `name`.

Remove the duplicate (incorrect) name.

Harbormaster completed remote builds in B72232: Diff 175585.Wed, Apr 15, 4:05 PM

michaelo edited the summary of this revision. (Show Details)Wed, Apr 15, 4:06 PM

michaelo marked an inline comment as done.

michaelo added inline comments.

sys/geom/label/g_label.c
255	Totally right!

markj accepted this revision.Wed, Apr 15, 4:07 PM

This revision is now accepted and ready to land.Wed, Apr 15, 4:07 PM

Closed by commit rG92c69c791c43: glabel(8): Fix debug message for already existing labels (authored by michaelo). · Explain WhyWed, Apr 15, 6:19 PM

This revision was automatically updated to reflect the committed changes.

michaelo marked an inline comment as done.

michaelo added a commit: rG92c69c791c43: glabel(8): Fix debug message for already existing labels.

Revision Contents
Changeset List

Path

Size

sys/

geom/

label/

g_label.c

4 lines

Diff 175585

View Options

glabel(8): Fix debug message for already existing labelsClosedPublicActions