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)
	Nov 5 2024, 12:48 AM

	Unknown Object (File)
	Oct 9 2024, 8:04 AM

	Unknown Object (File)
	Oct 9 2024, 5:09 AM

	Unknown Object (File)
	Oct 7 2024, 4:40 AM

	Unknown Object (File)
	Oct 6 2024, 6:44 PM

	Unknown Object (File)
	Oct 2 2024, 8:29 AM

	Unknown Object (File)
	Oct 1 2024, 2:59 AM

	Unknown Object (File)
	Sep 21 2024, 6:07 AM

View All 53 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