Clean up Warning:unused-but-set-variable spotted by gcc-4.9.
ClosedPublic
Actions

Authored by araujo on Jan 29 2016, 3:54 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Apr 16, 7:06 PM

	Unknown Object (File)
	Dec 20 2023, 12:04 AM

	Unknown Object (File)
	Aug 15 2023, 4:02 PM

	Unknown Object (File)
	Mar 15 2023, 3:44 PM

	Unknown Object (File)
	Mar 7 2023, 9:10 PM

	Unknown Object (File)
	Mar 6 2023, 3:59 AM

	Unknown Object (File)
	Feb 18 2023, 7:40 AM

	Unknown Object (File)
	Dec 31 2022, 6:37 AM

View All 32 Files

Subscribers

imp

Details

Reviewers

• rodrigc
slm
bapt
mav

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2305
Build 2314: arc lint + arc unit

Event Timeline

araujo updated this revision to Diff 12807.Jan 29 2016, 3:54 AM

araujo retitled this revision from to Clean up Warning:unused-but-set-variable spotted by gcc-4.9..

araujo updated this object.

araujo edited the test plan for this revision. (Show Details)

araujo added reviewers: • rodrigc, bapt, slm.

Herald added a subscriber: imp. · View Herald TranscriptJan 29 2016, 3:54 AM

OK, but get approval from mps experts.

This revision is now accepted and ready to land.Jan 29 2016, 4:34 AM

It looks fine to me, but it would be good to get comments from LSI/Avago (slm@).

Looks fine. I modified the mapping_get_dev_info function and neglected to remove the unused variable. The 'sg' variable has been there from the beginning it looks like, but ken@ made the changes to the SMP part.

slm accepted this revision.Jan 29 2016, 5:18 PM

slm edited edge metadata.

Committed: https://svnweb.freebsd.org/changeset/base/295113

Revision Contents
Changeset List

Path

Size

sys/

dev/

mps/

mps_mapping.c

4 lines

mps_sas.c

2 lines

Diff 12807

View Options

sys/dev/mps/mps_mapping.c