Skip to content

Commit

Permalink
ComplexMatrix2D and NumericMatrix2D both 'does numericmatrix' to show…
Browse files Browse the repository at this point in the history
… their similarities. Also, the NumMatrix2D tests should inherit from NumericMatrixTest, which adds several more tests for arithmetic routine. Changing this exposes some broken tests, but it's more honest
  • Loading branch information
Whiteknight committed Aug 18, 2010
1 parent 2836508 commit 2c6fd2b
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/pmc/complexmatrix2d.pmc
Expand Up @@ -386,7 +386,7 @@ intkey_to_coords(PARROT_INTERP, const INTVAL rows, const INTVAL cols,
}
}

pmclass ComplexMatrix2D dynpmc auto_attrs provides matrix {
pmclass ComplexMatrix2D dynpmc auto_attrs provides matrix provides numericmatrix {
ATTR FLOATVAL * storage;
ATTR INTVAL rows;
ATTR INTVAL cols;
Expand Down
2 changes: 1 addition & 1 deletion src/pmc/nummatrix2d.pmc
Expand Up @@ -238,7 +238,7 @@ floats_are_equal(FLOATVAL a, FLOATVAL b)
}


pmclass NumMatrix2D dynpmc auto_attrs provides matrix {
pmclass NumMatrix2D dynpmc auto_attrs provides matrix provides numericmatrix {
ATTR FLOATVAL * storage;
ATTR INTVAL rows;
ATTR INTVAL cols;
Expand Down
2 changes: 1 addition & 1 deletion t/pmc/nummatrix2d.t
@@ -1,4 +1,4 @@
class Test::NumMatrix2D is Pla::Matrix::MatrixTest;
class Test::NumMatrix2D is Pla::Matrix::NumericMatrixTest;

# Test boilerplate.

Expand Down

0 comments on commit 2c6fd2b

Please sign in to comment.