sx: avoid gcc -Wunused-value
ClosedPublic
Actions

Authored by rlibby on Jul 5 2024, 2:08 AM.

Details

Reviewers

imp
kib

Commits

rG430be7e1ee71: sx: avoid gcc -Wunused-value

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 58502
Build 55390: arc lint + arc unit

Event Timeline

rlibby requested review of this revision.Jul 5 2024, 2:08 AM

rlibby created this revision.

Harbormaster completed remote builds in B58502: Diff 140559.Jul 5 2024, 2:08 AM

Why didn't sx_lock() complain?

In D45875#1045918, @imp wrote:

Why didn't sx_lock() complain?

Oops. It did. I missed it. Will fix. Thanks.

imp: sx_xlock too

Also fixup whitespace while touching these lines.

Harbormaster completed remote builds in B58503: Diff 140560.Jul 5 2024, 4:24 AM

Then what about sx_try_xlock()?

In D45875#1045923, @kib wrote:

Then what about sx_try_xlock()?

Don't we always test whether that succeeded or not. Otherwise why try??

Yeah, we should use the value from sx_try_lock, and we'd like to enforce that it's checked too I guess.

Any more input on this one?

kib accepted this revision.Jul 9 2024, 6:28 PM

This revision is now accepted and ready to land.Jul 9 2024, 6:28 PM

Closed by commit rG430be7e1ee71: sx: avoid gcc -Wunused-value (authored by rlibby). · Explain WhyJul 9 2024, 6:46 PM

This revision was automatically updated to reflect the committed changes.

rlibby added a commit: rG430be7e1ee71: sx: avoid gcc -Wunused-value.

Revision Contents
Changeset List

Path

Size

sys/

sx.h

2 lines

Diff 140559

View Options

sx: avoid gcc -Wunused-valueClosedPublicActions