Skip to content

Commit

Permalink
Merge pull request #1159 from votjakovr/develop
Browse files Browse the repository at this point in the history
Fixed memory leak in GaussianLikelihood and StudentsTLikelihood unit tests
  • Loading branch information
karlnapf committed Jun 6, 2013
2 parents f2221e4 + 43cabcf commit 9850f44
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
5 changes: 5 additions & 0 deletions tests/unit/regression/gp/GaussianLikelihood_unittest.cc
Expand Up @@ -53,6 +53,7 @@ TEST(GaussianLikelihood,get_log_probability_f)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

TEST(GaussianLikelihood,get_log_probability_derivative_f)
Expand Down Expand Up @@ -107,6 +108,7 @@ TEST(GaussianLikelihood,get_log_probability_derivative_f)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

TEST(GaussianLikelihood,get_first_derivative)
Expand Down Expand Up @@ -152,6 +154,7 @@ TEST(GaussianLikelihood,get_first_derivative)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

TEST(GaussianLikelihood,get_second_derivative)
Expand Down Expand Up @@ -197,6 +200,7 @@ TEST(GaussianLikelihood,get_second_derivative)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

TEST(GaussianLikelihood,get_third_derivative)
Expand Down Expand Up @@ -242,6 +246,7 @@ TEST(GaussianLikelihood,get_third_derivative)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

#endif // HAVE_EIGEN3
5 changes: 5 additions & 0 deletions tests/unit/regression/gp/StudentsTLikelihood_unittest.cc
Expand Up @@ -53,6 +53,7 @@ TEST(StudentsTLikelihood,get_log_probability_f)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

TEST(StudentsTLikelihood,get_log_probability_derivative_f)
Expand Down Expand Up @@ -107,6 +108,7 @@ TEST(StudentsTLikelihood,get_log_probability_derivative_f)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

TEST(StudentsTLikelihood,get_first_derivative)
Expand Down Expand Up @@ -163,6 +165,7 @@ TEST(StudentsTLikelihood,get_first_derivative)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

TEST(StudentsTLikelihood,get_second_derivative)
Expand Down Expand Up @@ -219,6 +222,7 @@ TEST(StudentsTLikelihood,get_second_derivative)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

TEST(StudentsTLikelihood,get_third_derivative)
Expand Down Expand Up @@ -275,6 +279,7 @@ TEST(StudentsTLikelihood,get_third_derivative)

// clean up
SG_UNREF(likelihood);
SG_UNREF(labels);
}

#endif // HAVE_EIGEN3

0 comments on commit 9850f44

Please sign in to comment.