Bug 6043

Summary:	A pedantry patch (useful for gcc4)
Product:	dbus	Reporter:	Bernard Leak <bernard>
Component:	core	Assignee:	Havoc Pennington <hp>
Status:	RESOLVED FIXED	QA Contact:
Severity:	minor
Priority:	high	CC:	Woebbeking
Version:	unspecified
Hardware:	All
OS:	All
Whiteboard:
i915 platform:		i915 features:
Attachments:	Goodbye to two semicolons

Description Bernard Leak 2006-02-26 00:17:25 UTC

gcc4 gags (rightly) on superfluous ; in some places, at least with
warnings screwed up highly enough - that is, it produces an ERROR
and not just a warning.

This tiny patch improves the headers to remove two of them.
That still leaves plenty of warnings to go (notably lots of
"pointer targets differ in signedness"),

Comment 1 Bernard Leak 2006-02-26 00:18:05 UTC

Created attachment 4748 [details] [review]
Goodbye to two semicolons

Comment 2 Havoc Pennington 2006-02-27 04:52:34 UTC

Comment on attachment 4748 [details] [review]
Goodbye to two semicolons

Looks good, thanks.

Comment 3 André Wöbbeking 2006-06-03 07:40:33 UTC

and is still not applied :-(

Comment 4 Thiago Macieira 2006-06-12 02:21:17 UTC

Patch applied.
/cvs/dbus/dbus/dbus/dbus-arch-deps.h.in,v  <--  dbus/dbus-arch-deps.h.in
new revision: 1.4; previous revision: 1.3

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.