Paths

Table of Contentst

Differential D31948

mana: Cast an unused value to void to quiet a warning.
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Aug 12, 2:58 PM

	Unknown Object (File)
	Wed, Aug 5, 9:26 AM

	Unknown Object (File)
	Wed, Aug 5, 6:18 AM

	Unknown Object (File)
	Tue, Aug 4, 6:03 AM

	Unknown Object (File)
	Sat, Jul 25, 7:24 PM

	Unknown Object (File)
	Sat, Jul 25, 2:28 AM

	Unknown Object (File)
	Mon, Jul 20, 10:30 AM

	Unknown Object (File)
	Sun, Jul 19, 8:55 AM

Subscribers

Details

Reviewers

Commits

rG0def501d143a: mana: Cast an unused value to void to quiet a warning.

Summary

This appeases a -Wunused-value warning from GCC 9.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

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

Harbormaster completed remote builds in B41502: Diff 95074.Sep 13 2021, 6:28 PM

Adding an atomic_inc() wrapper might be a cleaner way of fixing this.

imp added inline comments.Sep 13 2021, 7:11 PM

sys/dev/mana/mana_en.c
579	why not just atmoic_inc?

lwhsu added a subscriber: lwhsu.Sep 14 2021, 2:13 AM

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

sys/dev/mana/mana_en.c
579	Because we don't have it yet and would need to add a wrapper in the foo.h compat header that defines atomic_inc_return(). I did suggest it as an alternative though if that would be cleaner.

Thanks for fixing this warning.

This revision is now accepted and ready to land.Sep 16 2021, 4:01 AM

Closed by commit rG0def501d143a: mana: Cast an unused value to void to quiet a warning. (authored by jhb). · Explain WhySep 25 2021, 6:31 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG0def501d143a: mana: Cast an unused value to void to quiet a warning..

Revision Contents
Changeset List

Path

Size

sys/

dev/

mana/

2 lines

Diff 95736

sys/dev/mana/mana_en.c

Loading...