Page MenuHomeFreeBSD
Differential D3473

GCC Warning - unused-but-set-variable
ClosedPublic
Actions

Authored by araujo on Aug 25 2015, 6:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 22 2026, 8:55 PM
Unknown Object (File)
Jan 3 2026, 1:06 AM
Unknown Object (File)
Nov 22 2025, 11:12 PM
Unknown Object (File)
Nov 20 2025, 5:33 PM
Unknown Object (File)
Nov 20 2025, 3:05 AM
Unknown Object (File)
Nov 5 2025, 4:03 AM
Unknown Object (File)
Oct 24 2025, 9:35 PM
Unknown Object (File)
Oct 22 2025, 8:48 PM
View All Files
Subscribers
imp

Details

Reviewers
• rodrigc
dim
melifaro
bapt
Commits
rS287130: Code cleanup unused-but-set-variable spotted by gcc.
Summary

Some code clean up.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

araujo updated this revision to Diff 8188.Aug 25 2015, 6:05 AM
araujo retitled this revision from to GCC Warning - unused-but-set-variable.
araujo updated this object.
araujo edited the test plan for this revision. (Show Details)
araujo added reviewers: • rodrigc, bapt.
Herald added a subscriber: imp. · View Herald TranscriptAug 25 2015, 6:05 AM
araujo updated this revision to Diff 8190.Aug 25 2015, 6:31 AM

Some more cleanup.

araujo added reviewers: melifaro, dim.Aug 25 2015, 6:34 AM
araujo mentioned this in D3474: GCC Warning - unused-but-set-variable.Aug 25 2015, 8:21 AM
araujo updated this object.
melifaro accepted this revision.Aug 25 2015, 8:51 AM
melifaro edited edge metadata.

Looks fine to me.

This revision is now accepted and ready to land.Aug 25 2015, 8:51 AM
bapt accepted this revision.Aug 25 2015, 9:45 AM
bapt edited edge metadata.
Closed by commit rS287130: Code cleanup unused-but-set-variable spotted by gcc. (authored by araujo). · Explain WhyAug 25 2015, 3:32 PM
This revision was automatically updated to reflect the committed changes.
araujo mentioned this in rS287130: Code cleanup unused-but-set-variable spotted by gcc..

Revision Contents
Changeset List

PathSize
head/
sbin/
ipfw/
4 lines
6 lines
8 lines

Diff 8200

View Options

head/sbin/ipfw/ipfw2.c

Loading...
View Options

head/sbin/ipfw/nat.c

Loading...
View Options

head/sbin/ipfw/tables.c

Loading...