Page MenuHomeFreeBSD

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

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

Details

Diff Detail

Repository
rS FreeBSD src repository
Lint
Lint OK
Unit
No Unit 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.
rodrigc accepted this revision.Jan 29 2016, 4:32 AM
rodrigc added a reviewer: mav.
rodrigc edited edge metadata.

OK, but get approval from mps experts.

This revision is now accepted and ready to land.Jan 29 2016, 4:34 AM
mav accepted this revision.Jan 29 2016, 10:34 AM
mav edited edge metadata.

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

slm edited edge metadata.Jan 29 2016, 5:17 PM

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.