Differential D26504

fix compiler warnings-as-errors on gcc6 in rcorder/crunchgen
ClosedPublic
Actions

Authored by adrian on Sep 21 2020, 6:33 AM.

Tags

None

Referenced Files

	F154995463: D26504.diff
	Thu, Apr 30, 1:46 PM

	Unknown Object (File)
	Thu, Apr 30, 11:41 AM

	Unknown Object (File)
	Tue, Apr 21, 7:46 AM

	Unknown Object (File)
	Tue, Apr 21, 7:46 AM

	Unknown Object (File)
	Mon, Apr 20, 8:57 AM

	Unknown Object (File)
	Sun, Apr 12, 6:44 PM

	Unknown Object (File)
	Sat, Apr 11, 12:45 AM

	Unknown Object (File)
	Fri, Apr 10, 4:28 PM

Subscribers

Details

Reviewers

imp
kevans
emaste

Commits

rS365957: [rcorder] [crunch] Fix C function declarations to include void

Summary

/usr/home/adrian/work/freebsd/head-embedded/src/sbin/rcorder/rcorder.c: In function 'generate_graphviz_header':
/usr/home/adrian/work/freebsd/head-embedded/src/sbin/rcorder/rcorder.c:993:1: error: old-style function definition [-Werror=old-style-definition]
generate_graphviz_header()
^~~~~~~~~~~~~~~~~~~~~~~~

Test Plan

Compiled using gcc-mips 6.4.0.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33685
Build 30923: arc lint + arc unit

Event Timeline

adrian created this revision.Sep 21 2020, 6:33 AM

Herald added a subscriber: imp. · View Herald TranscriptSep 21 2020, 6:33 AM

adrian requested review of this revision.Sep 21 2020, 6:33 AM

Harbormaster completed remote builds in B33685: Diff 77279.Sep 21 2020, 6:33 AM

ignore the pmc related bits, it's not relevant for this fix

Non pmc bits look good to go

usr.sbin/Makefile
189	N

This revision is now accepted and ready to land.Sep 21 2020, 6:46 AM

kevans accepted this revision.Sep 21 2020, 1:07 PM

pmc parts were changed in rS334749 when libpmc added a C++ file but this shouldn't be necessary

LGTM except the usr.sbin/Makefile change

emaste accepted this revision.Sep 21 2020, 1:56 PM

Closed by commit rS365957: [rcorder] [crunch] Fix C function declarations to include void (authored by adrian). · Explain WhySep 21 2020, 6:00 PM

This revision was automatically updated to reflect the committed changes.

adrian added a commit: rS365957: [rcorder] [crunch] Fix C function declarations to include void.

Revision Contents
Changeset List

Path

Size

sbin/

rcorder/

6 lines

usr.sbin/

6 lines

crunch/

crunchgen/

crunched_main.c

2 lines

pmccontrol/

2 lines

pmcstat/

2 lines

Diff 77279

sbin/rcorder/rcorder.c

Loading...

usr.sbin/Makefile

Loading...

usr.sbin/crunch/crunchgen/crunched_main.c

Loading...

usr.sbin/pmccontrol/Makefile

Loading...

usr.sbin/pmcstat/Makefile

Loading...