GNOME Bugzilla – Bug 318095
Compilation fixes
Last modified: 2006-08-07 21:17:45 UTC
Attached patches fixes fome compilation warnings. Used for version 2.12.0.
Created attachment 53097 [details] [review] gnome-utils-printf.patch Missing argument to g_snprintf.
Created attachment 53098 [details] [review] gnome-utils-decl.patch Fix of missing includes.
Created attachment 53099 [details] [review] gnome-utils-include.patch Another fix for missing includes.
Created attachment 53100 [details] [review] screen-shot-error-dlg.diff Report about insufficient permissions. Author: Shakti Prasad. https://bugzilla.novell.com/show_bug.cgi?id=72379
Created attachment 53101 [details] [review] gnome-utils-logview.diff Prevent gcc cast warning.
The first attachment (missing argument) was already fixed a few days ago by Rodrigo Moya. I committed patches #2, #3 and #5. I'd rather wait for approval from jrb before committing patch #4 (gnome-screenshot). Thanks a lot for all your work !
Created attachment 53115 [details] [review] gnome-utils-strict-aliasing.patch I have an another patch, which prevents "breaks strict aliasing rules" warnings (which can cause miscompilation) but causes "passing argument ... from incompatible pointer type" warnings. You may not want to submit it.
Looks like the last patch here is for 2.14.x? Is that right? At least only the part for gnome-screenshot applies here against HEAD.
*** Bug 324552 has been marked as a duplicate of this bug. ***
Created attachment 70387 [details] [review] updated patch I added some more compiler warning fixes. This patch contains the strict-aliasing warning fix from above that still applies to HEAD.
Fixed in HEAD and in 2.15.91. Thanks to Stanislav and to Kjartan.