diff mbox

[Branch,~glmark2-dev/glmark2/trunk] Rev 107: Fix compiler warning about potentially uninitialized variable.

Message ID 20110721123637.17019.6749.launchpad@loganberry.canonical.com
State Accepted
Headers show

Commit Message

Jesse Barker July 21, 2011, 12:36 p.m. UTC
------------------------------------------------------------
revno: 107
committer: Jesse Barker <jesse.barker@linaro.org>
timestamp: Tue 2011-07-12 11:22:45 -0700
message:
  Fix compiler warning about potentially uninitialized variable.
  Add '-Werror' to CXXFLAGS so that this sort of thing generates an error.
modified:
  src/model.cpp
  wscript


--
lp:glmark2
https://code.launchpad.net/~glmark2-dev/glmark2/trunk

You are subscribed to branch lp:glmark2.
To unsubscribe from this branch go to https://code.launchpad.net/~glmark2-dev/glmark2/trunk/+edit-subscription
diff mbox

Patch

=== modified file 'src/model.cpp'
--- src/model.cpp	2011-07-12 09:05:53 +0000
+++ src/model.cpp	2011-07-12 18:22:45 +0000
@@ -160,7 +160,7 @@ 
 bool
 Model::load_3ds(const std::string &filename)
 {
-    Object *object;
+    Object *object(0);
 
     Log::debug("Loading model from 3ds file '%s'\n", filename.c_str());
 

=== modified file 'wscript'
--- wscript	2011-06-30 14:13:04 +0000
+++ wscript	2011-07-12 18:22:45 +0000
@@ -62,7 +62,7 @@ 
         ctx.check_cfg(package = pkg, uselib_store = uselib,
                       args = '--cflags --libs', mandatory = mandatory)
 
-    ctx.env.append_unique('CXXFLAGS', '-Wall -Wextra -fms-extensions'.split(' '))
+    ctx.env.append_unique('CXXFLAGS', '-Wall -Werror -Wextra -fms-extensions'.split(' '))
 
     # Prepend -O# and -g flags so that they can be overriden by the
     # CFLAGS environment variable