]> gitweb.michael.orlitzky.com - spline3.git/commitdiff
Fix some warnings and hlint suggestions.
authorMichael Orlitzky <michael@orlitzky.com>
Sun, 8 May 2011 20:06:38 +0000 (16:06 -0400)
committerMichael Orlitzky <michael@orlitzky.com>
Sun, 8 May 2011 20:06:38 +0000 (16:06 -0400)
src/Cardinal.hs
src/Tests/Cube.hs
src/Tetrahedron.hs

index 42b1260bb13f72f530bd725fd89943da1c1d004f..9a5d025efa9c43537aba416491b4fb10fa8e2494 100644 (file)
@@ -54,7 +54,7 @@ instance Num Cardinal where
     x + y = Sum x y
     x - y = Difference x y
     x * y = Product x y
-    negate x = Product (Scalar (-1)) x
+    negate = Product (Scalar (-1))
     abs x = x
     signum x = x
     fromInteger x = Scalar (fromIntegral x)
@@ -64,7 +64,7 @@ instance Num Cardinal where
 --   take quotients of directions.
 instance Fractional Cardinal where
     x / y   = Quotient x y
-    recip x = Quotient (Scalar 1) x
+    recip = Quotient (Scalar 1)
     fromRational x = Scalar (fromRational x)
 
 
index 888c48f4840d54ec62f95f0f3e4aa0fe17519e7c..11159b5bc6f668a6e14f0976646c5e50398d1236 100644 (file)
@@ -8,7 +8,6 @@ import Cube
 import FunctionValues (FunctionValues)
 import Tests.FunctionValues ()
 import Tetrahedron (b0, b1, b2, b3, c,
-                    Tetrahedron(Tetrahedron),
                     v0, v1, v2, v3, volume)
 
 instance Arbitrary Cube where
@@ -30,10 +29,10 @@ instance Arbitrary Cube where
 --   (which comprise cubes of positive volume) must have positive volume
 --   as well.
 prop_all_volumes_positive :: Cube -> Bool
-prop_all_volumes_positive c =
+prop_all_volumes_positive cube =
     null nonpositive_volumes
     where
-      ts = tetrahedrons c
+      ts = tetrahedrons cube
       volumes = map volume ts
       nonpositive_volumes = filter (<= 0) volumes
 
@@ -41,225 +40,225 @@ prop_all_volumes_positive c =
 --   already know their volumes. There's 24 tetrahedra to a cube, so
 --   we'd expect the volume of each one to be (1/24)*h^3.
 prop_tetrahedron0_volumes_exact :: Cube -> Bool
-prop_tetrahedron0_volumes_exact c =
-    volume (tetrahedron0 c) ~= (1/24)*(delta^(3::Int))
+prop_tetrahedron0_volumes_exact cube =
+    volume (tetrahedron0 cube) ~= (1/24)*(delta^(3::Int))
     where
-      delta = h c
+      delta = h cube
 
 -- | In fact, since all of the tetrahedra are identical, we should
 --   already know their volumes. There's 24 tetrahedra to a cube, so
 --   we'd expect the volume of each one to be (1/24)*h^3.
 prop_tetrahedron1_volumes_exact :: Cube -> Bool
-prop_tetrahedron1_volumes_exact c =
-    volume (tetrahedron1 c) ~= (1/24)*(delta^(3::Int))
+prop_tetrahedron1_volumes_exact cube =
+    volume (tetrahedron1 cube) ~= (1/24)*(delta^(3::Int))
     where
-      delta = h c
+      delta = h cube
 
 -- | In fact, since all of the tetrahedra are identical, we should
 --   already know their volumes. There's 24 tetrahedra to a cube, so
 --   we'd expect the volume of each one to be (1/24)*h^3.
 prop_tetrahedron2_volumes_exact :: Cube -> Bool
-prop_tetrahedron2_volumes_exact c =
-    volume (tetrahedron2 c) ~= (1/24)*(delta^(3::Int))
+prop_tetrahedron2_volumes_exact cube =
+    volume (tetrahedron2 cube) ~= (1/24)*(delta^(3::Int))
     where
-      delta = h c
+      delta = h cube
 
 -- | In fact, since all of the tetrahedra are identical, we should
 --   already know their volumes. There's 24 tetrahedra to a cube, so
 --   we'd expect the volume of each one to be (1/24)*h^3.
 prop_tetrahedron3_volumes_exact :: Cube -> Bool
-prop_tetrahedron3_volumes_exact c =
-    volume (tetrahedron3 c) ~= (1/24)*(delta^(3::Int))
+prop_tetrahedron3_volumes_exact cube =
+    volume (tetrahedron3 cube) ~= (1/24)*(delta^(3::Int))
     where
-      delta = h c
+      delta = h cube
 
 -- | In fact, since all of the tetrahedra are identical, we should
 --   already know their volumes. There's 24 tetrahedra to a cube, so
 --   we'd expect the volume of each one to be (1/24)*h^3.
 prop_tetrahedron4_volumes_exact :: Cube -> Bool
-prop_tetrahedron4_volumes_exact c =
-    volume (tetrahedron4 c) ~= (1/24)*(delta^(3::Int))
+prop_tetrahedron4_volumes_exact cube =
+    volume (tetrahedron4 cube) ~= (1/24)*(delta^(3::Int))
     where
-      delta = h c
+      delta = h cube
 
 -- | In fact, since all of the tetrahedra are identical, we should
 --   already know their volumes. There's 24 tetrahedra to a cube, so
 --   we'd expect the volume of each one to be (1/24)*h^3.
 prop_tetrahedron5_volumes_exact :: Cube -> Bool
-prop_tetrahedron5_volumes_exact c =
-    volume (tetrahedron5 c) ~= (1/24)*(delta^(3::Int))
+prop_tetrahedron5_volumes_exact cube =
+    volume (tetrahedron5 cube) ~= (1/24)*(delta^(3::Int))
     where
-      delta = h c
+      delta = h cube
 
 -- | In fact, since all of the tetrahedra are identical, we should
 --   already know their volumes. There's 24 tetrahedra to a cube, so
 --   we'd expect the volume of each one to be (1/24)*h^3.
 prop_tetrahedron6_volumes_exact :: Cube -> Bool
-prop_tetrahedron6_volumes_exact c =
-    volume (tetrahedron6 c) ~= (1/24)*(delta^(3::Int))
+prop_tetrahedron6_volumes_exact cube =
+    volume (tetrahedron6 cube) ~= (1/24)*(delta^(3::Int))
     where
-      delta = h c
+      delta = h cube
 
 -- | In fact, since all of the tetrahedra are identical, we should
 --   already know their volumes. There's 24 tetrahedra to a cube, so
 --   we'd expect the volume of each one to be (1/24)*h^3.
 prop_tetrahedron7_volumes_exact :: Cube -> Bool
-prop_tetrahedron7_volumes_exact c =
-    volume (tetrahedron7 c) ~= (1/24)*(delta^(3::Int))
+prop_tetrahedron7_volumes_exact cube =
+    volume (tetrahedron7 cube) ~= (1/24)*(delta^(3::Int))
     where
-      delta = h c
+      delta = h cube
 
 -- | All tetrahedron should have their v0 located at the center of the cube.
 prop_v0_all_equal :: Cube -> Bool
-prop_v0_all_equal c = (v0 t0) == (v0 t1)
+prop_v0_all_equal cube = (v0 t0) == (v0 t1)
     where
-      t0 = head (tetrahedrons c) -- Doesn't matter which two we choose.
-      t1 = head $ tail (tetrahedrons c)
+      t0 = head (tetrahedrons cube) -- Doesn't matter which two we choose.
+      t1 = head $ tail (tetrahedrons cube)
 
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron0_volumes_positive :: Cube -> Bool
-prop_tetrahedron0_volumes_positive c =
-    volume (tetrahedron0 c) > 0
+prop_tetrahedron0_volumes_positive cube =
+    volume (tetrahedron0 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron1_volumes_positive :: Cube -> Bool
-prop_tetrahedron1_volumes_positive c =
-    volume (tetrahedron1 c) > 0
+prop_tetrahedron1_volumes_positive cube =
+    volume (tetrahedron1 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron2_volumes_positive :: Cube -> Bool
-prop_tetrahedron2_volumes_positive c =
-    volume (tetrahedron2 c) > 0
+prop_tetrahedron2_volumes_positive cube =
+    volume (tetrahedron2 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron3_volumes_positive :: Cube -> Bool
-prop_tetrahedron3_volumes_positive c =
-    volume (tetrahedron3 c) > 0
+prop_tetrahedron3_volumes_positive cube =
+    volume (tetrahedron3 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron4_volumes_positive :: Cube -> Bool
-prop_tetrahedron4_volumes_positive c =
-    volume (tetrahedron4 c) > 0
+prop_tetrahedron4_volumes_positive cube =
+    volume (tetrahedron4 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron5_volumes_positive :: Cube -> Bool
-prop_tetrahedron5_volumes_positive c =
-    volume (tetrahedron5 c) > 0
+prop_tetrahedron5_volumes_positive cube =
+    volume (tetrahedron5 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron6_volumes_positive :: Cube -> Bool
-prop_tetrahedron6_volumes_positive c =
-    volume (tetrahedron6 c) > 0
+prop_tetrahedron6_volumes_positive cube =
+    volume (tetrahedron6 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron7_volumes_positive :: Cube -> Bool
-prop_tetrahedron7_volumes_positive c =
-    volume (tetrahedron7 c) > 0
+prop_tetrahedron7_volumes_positive cube =
+    volume (tetrahedron7 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron8_volumes_positive :: Cube -> Bool
-prop_tetrahedron8_volumes_positive c =
-    volume (tetrahedron8 c) > 0
+prop_tetrahedron8_volumes_positive cube =
+    volume (tetrahedron8 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron9_volumes_positive :: Cube -> Bool
-prop_tetrahedron9_volumes_positive c =
-    volume (tetrahedron9 c) > 0
+prop_tetrahedron9_volumes_positive cube =
+    volume (tetrahedron9 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron10_volumes_positive :: Cube -> Bool
-prop_tetrahedron10_volumes_positive c =
-    volume (tetrahedron10 c) > 0
+prop_tetrahedron10_volumes_positive cube =
+    volume (tetrahedron10 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron11_volumes_positive :: Cube -> Bool
-prop_tetrahedron11_volumes_positive c =
-    volume (tetrahedron11 c) > 0
+prop_tetrahedron11_volumes_positive cube =
+    volume (tetrahedron11 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron12_volumes_positive :: Cube -> Bool
-prop_tetrahedron12_volumes_positive c =
-    volume (tetrahedron12 c) > 0
+prop_tetrahedron12_volumes_positive cube =
+    volume (tetrahedron12 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron13_volumes_positive :: Cube -> Bool
-prop_tetrahedron13_volumes_positive c =
-    volume (tetrahedron13 c) > 0
+prop_tetrahedron13_volumes_positive cube =
+    volume (tetrahedron13 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron14_volumes_positive :: Cube -> Bool
-prop_tetrahedron14_volumes_positive c =
-    volume (tetrahedron14 c) > 0
+prop_tetrahedron14_volumes_positive cube =
+    volume (tetrahedron14 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron15_volumes_positive :: Cube -> Bool
-prop_tetrahedron15_volumes_positive c =
-    volume (tetrahedron15 c) > 0
+prop_tetrahedron15_volumes_positive cube =
+    volume (tetrahedron15 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron16_volumes_positive :: Cube -> Bool
-prop_tetrahedron16_volumes_positive c =
-    volume (tetrahedron16 c) > 0
+prop_tetrahedron16_volumes_positive cube =
+    volume (tetrahedron16 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron17_volumes_positive :: Cube -> Bool
-prop_tetrahedron17_volumes_positive c =
-    volume (tetrahedron17 c) > 0
+prop_tetrahedron17_volumes_positive cube =
+    volume (tetrahedron17 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron18_volumes_positive :: Cube -> Bool
-prop_tetrahedron18_volumes_positive c =
-    volume (tetrahedron18 c) > 0
+prop_tetrahedron18_volumes_positive cube =
+    volume (tetrahedron18 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron19_volumes_positive :: Cube -> Bool
-prop_tetrahedron19_volumes_positive c =
-    volume (tetrahedron19 c) > 0
+prop_tetrahedron19_volumes_positive cube =
+    volume (tetrahedron19 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron20_volumes_positive :: Cube -> Bool
-prop_tetrahedron20_volumes_positive c =
-    volume (tetrahedron20 c) > 0
+prop_tetrahedron20_volumes_positive cube =
+    volume (tetrahedron20 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron21_volumes_positive :: Cube -> Bool
-prop_tetrahedron21_volumes_positive c =
-    volume (tetrahedron21 c) > 0
+prop_tetrahedron21_volumes_positive cube =
+    volume (tetrahedron21 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron22_volumes_positive :: Cube -> Bool
-prop_tetrahedron22_volumes_positive c =
-    volume (tetrahedron22 c) > 0
+prop_tetrahedron22_volumes_positive cube =
+    volume (tetrahedron22 cube) > 0
 
 -- | This pretty much repeats the prop_all_volumes_positive property,
 --   but will let me know which tetrahedrons's vertices are disoriented.
 prop_tetrahedron23_volumes_positive :: Cube -> Bool
-prop_tetrahedron23_volumes_positive c =
-    volume (tetrahedron23 c) > 0
+prop_tetrahedron23_volumes_positive cube =
+    volume (tetrahedron23 cube) > 0
 
 
 -- | Given in Sorokina and Zeilfelder, p. 79. Note that the third and
index 81fe91c11772122d0643b884e4fc3aea52163a43..85ac4a372d222cea1eaa56a821f54cf340e08b82 100644 (file)
@@ -198,7 +198,7 @@ c _ _ _ _ _ = error "coefficient index out of bounds"
 
 
 vol_matrix :: Tetrahedron -> Matrix Double
-vol_matrix t = (4><4) $
+vol_matrix t = (4><4)
                [1,  1,  1,  1,
                 x1, x2, x3, x4,
                 y1, y2, y3, y4,