From 18025123713f1fd45624819fbddb5ae20e18ce14 Mon Sep 17 00:00:00 2001 From: Joel Holdsworth Date: Mon, 29 Dec 2008 20:41:47 +0000 Subject: [PATCH] Reinstated -Werror and fixed the ftruncate warning --- src/backend/Makefile.am | 2 +- src/backend/filedescriptor.c | 3 ++- src/gui/Makefile.am | 2 +- src/lib/Makefile.am | 2 +- src/proc/Makefile.am | 16 ++++++++-------- 5 files changed, 13 insertions(+), 12 deletions(-) diff --git a/src/backend/Makefile.am b/src/backend/Makefile.am index 982b8e150..75bcc0d7e 100644 --- a/src/backend/Makefile.am +++ b/src/backend/Makefile.am @@ -19,7 +19,7 @@ liblumierabackend_la_srcdir = $(top_srcdir)/src/backend lib_LTLIBRARIES += liblumierabackend.la liblumierabackend_la_CPPFLAGS = $(AM_CPPFLAGS) -liblumierabackend_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wextra -Wall #-Werror +liblumierabackend_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wextra -Wall -Werror liblumierabackend_la_LIBADD = liblumiera.la liblumierabackend_la_SOURCES = \ diff --git a/src/backend/filedescriptor.c b/src/backend/filedescriptor.c index 906fdc3cd..846ec5d3f 100644 --- a/src/backend/filedescriptor.c +++ b/src/backend/filedescriptor.c @@ -299,7 +299,8 @@ lumiera_filedescriptor_delete (LumieraFiledescriptor self, const char* name) { TRACE (filedescriptor, "truncate %s to %lld", name, self->realsize); lumiera_filehandlecache_checkout (lumiera_fhcache, self->handle); - (void) ftruncate (lumiera_filehandle_handle (self->handle), self->realsize); + int dummy = ftruncate (lumiera_filehandle_handle (self->handle), self->realsize); + (void) dummy; /* this is present to silence a warning */ lumiera_filehandlecache_checkin (lumiera_fhcache, self->handle); } diff --git a/src/gui/Makefile.am b/src/gui/Makefile.am index 352cd28b0..467521430 100644 --- a/src/gui/Makefile.am +++ b/src/gui/Makefile.am @@ -84,7 +84,7 @@ gtk_gui_la_SOURCES = \ # noinst_LTLIBRARIES += libgui.la -libgui_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +libgui_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror libgui_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra libgui_la_CPPFLAGS = $(AM_CPPFLAGS) \ diff --git a/src/lib/Makefile.am b/src/lib/Makefile.am index ff83e55d4..834681720 100644 --- a/src/lib/Makefile.am +++ b/src/lib/Makefile.am @@ -18,7 +18,7 @@ liblumiera_la_srcdir = $(top_srcdir)/src/lib lib_LTLIBRARIES += liblumiera.la -liblumiera_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumiera_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumiera_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumiera_la_SOURCES = \ diff --git a/src/proc/Makefile.am b/src/proc/Makefile.am index 59ccd0c92..c0058adfa 100644 --- a/src/proc/Makefile.am +++ b/src/proc/Makefile.am @@ -21,7 +21,7 @@ liblumiproc_la_srcdir = $(top_srcdir)/src/proc noinst_LTLIBRARIES += liblumiproc.la -liblumiproc_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumiproc_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumiproc_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumiproc_la_SOURCES = \ @@ -36,7 +36,7 @@ liblumiproc_la_SOURCES = \ liblumiprocasset_la_srcdir = $(top_srcdir)/src/proc/asset noinst_LTLIBRARIES += liblumiprocasset.la -liblumiprocasset_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumiprocasset_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumiprocasset_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumiprocasset_la_SOURCES = \ @@ -61,7 +61,7 @@ liblumiprocasset_la_SOURCES = \ liblumiprocengine_la_srcdir = $(top_srcdir)/src/proc/engine noinst_LTLIBRARIES += liblumiprocengine.la -liblumiprocengine_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumiprocengine_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumiprocengine_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumiprocengine_la_SOURCES = \ @@ -84,7 +84,7 @@ liblumiprocengine_la_SOURCES = \ liblumiprocmobject_la_srcdir = $(top_srcdir)/src/proc/mobject noinst_LTLIBRARIES += liblumiprocmobject.la -liblumiprocmobject_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumiprocmobject_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumiprocmobject_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumiprocmobject_la_SOURCES = \ @@ -100,7 +100,7 @@ liblumiprocmobject_la_SOURCES = \ liblumiprocmobjectbuilder_la_srcdir = $(top_srcdir)/src/proc/mobject/builder noinst_LTLIBRARIES += liblumiprocmobjectbuilder.la -liblumiprocmobjectbuilder_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumiprocmobjectbuilder_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumiprocmobjectbuilder_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumiprocmobjectbuilder_la_SOURCES = \ @@ -114,7 +114,7 @@ liblumiprocmobjectbuilder_la_SOURCES = \ liblumiproccontrol_la_srcdir = $(top_srcdir)/src/proc/control noinst_LTLIBRARIES += liblumiproccontrol.la -liblumiproccontrol_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumiproccontrol_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumiproccontrol_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumiproccontrol_la_SOURCES = \ @@ -125,7 +125,7 @@ liblumiproccontrol_la_SOURCES = \ liblumiprocmobjectsession_la_srcdir = $(top_srcdir)/src/proc/mobject/session noinst_LTLIBRARIES += liblumiprocmobjectsession.la -liblumiprocmobjectsession_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumiprocmobjectsession_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumiprocmobjectsession_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumiprocmobjectsession_la_SOURCES = \ @@ -231,7 +231,7 @@ noinst_HEADERS += \ liblumieraproc_la_srcdir = $(top_srcdir)/src/proc lib_LTLIBRARIES += liblumieraproc.la -liblumieraproc_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra #-Werror +liblumieraproc_la_CFLAGS = $(AM_CFLAGS) -std=gnu99 -Wall -Wextra -Werror liblumieraproc_la_CXXFLAGS = $(AM_CXXFLAGS) -Wall -Wextra liblumieraproc_la_SOURCES =