From: David Wells Date: Sat, 5 Nov 2016 14:14:31 +0000 (-0400) Subject: Remove unused method arguments. X-Git-Tag: v8.5.0-rc1~455^2~2 X-Git-Url: https://gitweb.dealii.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1dffe66e08bb4a6d620b05d7a7bca5a2799538bb;p=dealii.git Remove unused method arguments. This is a protected method which is always called with two member variables with identical names as the function arguments. This was caught by PVS studio. --- diff --git a/include/deal.II/lac/precondition_block.h b/include/deal.II/lac/precondition_block.h index bf168e3afb..76eca4f604 100644 --- a/include/deal.II/lac/precondition_block.h +++ b/include/deal.II/lac/precondition_block.h @@ -212,8 +212,7 @@ protected: /** * Replacement of invert_diagblocks() for permuted preconditioning. */ - void invert_permuted_diagblocks(const std::vector &permutation, - const std::vector &inverse_permutation); + void invert_permuted_diagblocks(); public: /** * Deletes the inverse diagonal block matrices if existent, sets the diff --git a/include/deal.II/lac/precondition_block.templates.h b/include/deal.II/lac/precondition_block.templates.h index f8e9f65094..e6ced51b02 100644 --- a/include/deal.II/lac/precondition_block.templates.h +++ b/include/deal.II/lac/precondition_block.templates.h @@ -93,7 +93,7 @@ void PreconditionBlock::initialize if (parameters.invert_diagonal) { if (permutation.size() == M.m()) - invert_permuted_diagblocks(permutation, inverse_permutation); + invert_permuted_diagblocks(); else invert_diagblocks(); } @@ -112,9 +112,7 @@ void PreconditionBlock::initialize } template -void PreconditionBlock::invert_permuted_diagblocks -(const std::vector &permutation, - const std::vector &inverse_permutation) +void PreconditionBlock::invert_permuted_diagblocks() { Assert (A!=0, ExcNotInitialized()); Assert (blocksize!=0, ExcNotInitialized());