removed many warning when we activate them.

theano/compile/tests/test_debugmode.py

@@ -72,7 +72,7 @@ class BROKEN_ON_PURPOSE_StructuredDotCSC(gof.Op):
             || (%(z)s->dimensions[1] != %(b)s->dimensions[1])
             )
         {
-            if (%(z)s) Py_DECREF(%(z)s);
+            {Py_XDECREF(%(z)s);}
             npy_intp dims[] = {0,0};
             dims[0] = ((npy_int32 *)%(a_nrows)s->data)[0];
             dims[1] = %(b)s->dimensions[1];
@@ -189,13 +189,13 @@ class WeirdBrokenOp(gof.Op):
     def c_code(self, node, name, (a,), (z,), sub):
         if "inplace" in self.behaviour:
             z_code = """
-            if (%(z)s) Py_DECREF(%(z)s);
+            {Py_XDECREF(%(z)s);}
             Py_INCREF(%(a)s);
             %(z)s = %(a)s;
             """
         else:
             z_code = """
-            if (%(z)s) Py_DECREF(%(z)s);
+            {Py_XDECREF(%(z)s);}
             %(z)s = (PyArrayObject*) PyArray_SimpleNew(1, %(a)s->dimensions, %(a)s->descr->type_num);
             """
         prep_vars = """

theano/gof/cc.py

@@ -153,9 +153,9 @@ def struct_gen(args, struct_builders, blocks, sub):
             PyList_SET_ITEM(__ERROR, 0, err_type);
             PyList_SET_ITEM(__ERROR, 1, err_msg);
             PyList_SET_ITEM(__ERROR, 2, err_traceback);
-            Py_XDECREF(old_err_type);
-            Py_XDECREF(old_err_msg);
-            Py_XDECREF(old_err_traceback);
+            {Py_XDECREF(old_err_type);}
+            {Py_XDECREF(old_err_msg);}
+            {Py_XDECREF(old_err_traceback);}
         }
         // The failure code is returned to index what code block failed.
         return %(failure_var)s;
@@ -237,7 +237,7 @@ def get_c_extract(r, name, sub):
 def get_c_cleanup(r, name, sub):
     """WRITEME"""
     post = """
-    Py_XDECREF(py_%(name)s);
+    {Py_XDECREF(py_%(name)s);}
     """ % locals()
     return r.type.c_cleanup(name, sub) + post
 
@@ -249,7 +249,7 @@ def get_c_sync(r, name, sub):
       PyObject* old = PyList_GET_ITEM(storage_%(name)s, 0);
       {Py_XINCREF(py_%(name)s);}
       PyList_SET_ITEM(storage_%(name)s, 0, py_%(name)s);
-      Py_XDECREF(old);
+      {Py_XDECREF(old);}
     }
     """ % dict(sync = r.type.c_sync(name, sub), name = name, **sub)
 

theano/sandbox/conv.py

@@ -522,9 +522,8 @@ if ((!%(z)s)
   || (%(z)s->dimensions[3] != dim_zz[1])
   )
 {
-  if (%(z)s) Py_DECREF(%(z)s);
+  {Py_XDECREF(%(z)s);}
   npy_intp dims[4] = {0,0,0,0};
-  if(!dims) %(fail)s;
   dims[0]=%(self_bsize)s;
   dims[1]=%(self_nkern)s;
   dims[2]=dim_zz[0];
@@ -751,7 +750,7 @@ if ((!%(z)s)
   || (%(z)s->dimensions[3] != dim_zz[1])
   )
 {
-  if (%(z)s) Py_DECREF(%(z)s);
+  {Py_XDECREF(%(z)s);}
   npy_intp dims[4] = {0,0,0,0};
   dims[0]=%(self_bsize)s;
   dims[1]=%(self_nkern)s;
@@ -1007,9 +1006,8 @@ if ((!%(z)s)
   || (%(z)s->dimensions[3] != dim_zz[1])
   )
 {
-  if (%(z)s) Py_DECREF(%(z)s);
+  {Py_XDECREF(%(z)s);}
   npy_intp dims[4] = {0,0,0,0};
-  if(!dims) %(fail)s;
   dims[0]=%(self_bsize)s;
   dims[1]=%(self_nkern)s;
   dims[2]=dim_zz[0];

theano/sparse/basic.py

@@ -819,7 +819,7 @@ class StructuredDotCSC(gof.Op):
             || (%(z)s->dimensions[1] != %(b)s->dimensions[1])
             )
         {
-            if (%(z)s) Py_DECREF(%(z)s);
+            {Py_XDECREF(%(z)s)};
             npy_intp dims[] = {0,0};
             dims[0] = ((npy_int32 *)%(a_nrows)s->data)[0];
             dims[1] = %(b)s->dimensions[1];
@@ -951,7 +951,7 @@ class StructuredDotCSR(gof.Op):
             || (%(z)s->dimensions[1] != %(b)s->dimensions[1])       //b's columns
             )
         {
-            if (%(z)s) Py_DECREF(%(z)s);
+            {Py_XDECREF(%(z)s);}
             npy_intp dims[] = {0,0};
             dims[0] = %(a_ptr)s->dimensions[0]-1;
             dims[1] = %(b)s->dimensions[1];