Differential D31946

posixshmtest: Fix various warnings raised by GCC.
ClosedPublic
Actions

Authored by jhb on Sep 13 2021, 6:26 PM.

Details

Reviewers

khng
kib

Commits

rGcd16a848d1b3: posixshmtest: Fix various warnings raised by GCC.

Summary

Remove unused format string arguments.

Remove a set but unused variable.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 41500
Build 38389: arc lint + arc unit

Event Timeline

jhb created this revision.Sep 13 2021, 6:26 PM

Herald added a subscriber: imp. · View Herald TranscriptSep 13 2021, 6:26 PM

jhb requested review of this revision.Sep 13 2021, 6:26 PM

Harbormaster completed remote builds in B41500: Diff 95072.Sep 13 2021, 6:26 PM

kib accepted this revision.Sep 13 2021, 6:52 PM

kib added inline comments.

tests/sys/posixshm/posixshm_test.c
1108	Perhaps idea was to print out errno.

This revision is now accepted and ready to land.Sep 13 2021, 6:52 PM

khng accepted this revision.Sep 13 2021, 7:31 PM

jhb added inline comments.Sep 15 2021, 3:55 PM

tests/sys/posixshm/posixshm_test.c
1108	So I did that initially, but check_content_dealloc() doesn't always set errno when it fails, so I believe this was likely a copy-paste error.

Closed by commit rGcd16a848d1b3: posixshmtest: Fix various warnings raised by GCC. (authored by jhb). · Explain WhySep 15 2021, 4:06 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGcd16a848d1b3: posixshmtest: Fix various warnings raised by GCC..

Revision Contents
Changeset List

Path

Size

tests/

sys/

posixshm/

posixshm_test.c

16 lines

Diff 95072

View Options

tests/sys/posixshm/posixshm_test.c

posixshmtest: Fix various warnings raised by GCC.ClosedPublicActions