Fix various warnings with higher WARNS.
ClosedPublic
Actions

Authored by jhb on Oct 13 2020, 6:58 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Dec 11, 7:24 PM

	Unknown Object (File)
	Wed, Dec 11, 12:11 PM

	Unknown Object (File)
	Nov 14 2024, 6:50 AM

	Unknown Object (File)
	Nov 3 2024, 2:06 PM

	Unknown Object (File)
	Oct 17 2024, 12:32 AM

	Unknown Object (File)
	Oct 16 2024, 3:56 AM

	Unknown Object (File)
	Oct 16 2024, 3:56 AM

	Unknown Object (File)
	Sep 30 2024, 12:20 PM

View All 27 Files

Subscribers

imp

Details

Reviewers

markj
kevans

Commits

rS366687: Fix various warnings with higher WARNS.

Summary

Rename global 'crid' to 'requested_crid' to avoid shadowing.
Remove some unused function arguments.
Use __DECONST().

Test Plan

tried to use this this week and it failed to build, presumably due to WARNS' default changing to 6 globally

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 34138
Build 31298: arc lint + arc unit

Event Timeline

jhb created this revision.Oct 13 2020, 6:58 PM

Herald added a subscriber: imp. · View Herald TranscriptOct 13 2020, 6:58 PM

jhb requested review of this revision.Oct 13 2020, 6:58 PM

Harbormaster completed remote builds in B34138: Diff 78180.Oct 13 2020, 6:58 PM

kevans accepted this revision.Oct 13 2020, 7:22 PM

This revision is now accepted and ready to land.Oct 13 2020, 7:22 PM

markj accepted this revision.Oct 13 2020, 7:45 PM

jhb closed this revision.Oct 13 2020, 10:20 PM

jhb added a commit: rS366687: Fix various warnings with higher WARNS..

Revision Contents
Changeset List

Path

Size

tools/

crypto/

cryptocheck.c

117 lines

Diff 78180

View Options

tools/tools/crypto/cryptocheck.c

Fix various warnings with higher WARNS.ClosedPublicActions