Fred answered question about dot22scalar test

theano/tensor/tests/test_blas.py

@@ -601,13 +601,12 @@ def test_dot22scalar():
     f = theano.function([a,b,c],0.2*c *a*T.dot(a,b),mode=mode_blas_opt)
     topo = f.maker.env.toposort()
     #currently the canonizer don't always merge all Mul together...
-    #that force the optimizer to make a recursive search witch it don't do now.
-    #but it do it for 1 level of recursion.
+    # dot22scalar optimizer does not do a recursive search
+    # therefore, it doesn't find potential matches of the scalar.
+    # TODO: combine with the 'canonicalization' that is part of the Gemm optimizer.
+    #
     #    assert _dot22scalar in [x.op for x in topo]
     #    assert len(topo)==2
-    ### Fred, 
-    ### What are you talking about?
-    ### -James (March 28 2010)
     
     f(av,bv,cv)
     f = theano.function([a,b,c],c * a*0.2*T.dot(a,b),mode=m2)