[Dune-devel] [Dune-Commit] [Commit] dune-common - e5ca9a3: [release] Removes unused parameter warnings with -Wall -Wextra

Oliver Sander sander at igpm.rwth-aachen.de
Mon Dec 16 07:53:59 CET 2013


Hi Markus,
isn't this why we have DUNE_UNUSED?  It makes for much better reading IMHO.
Best,
Oliver

Am 13.12.2013 21:30, schrieb Markus Blatt:
> New commit, appeared at Fri Dec 13 21:30:33 2013 +0100
> as part of the following ref changes:
> 
>     branch refs/heads/master    updated from 6701492 -> e5ca9a3
> 
> Browsable version: http://cgit.dune-project.org/repositories/dune-common/commit/?id=e5ca9a37f6d7d6432c7219e85875844c60056cd2
> 
> ======================================================================
> 
> commit e5ca9a37f6d7d6432c7219e85875844c60056cd2
> Author: Markus Blatt <markus at dr-blatt.de>
> Date:   Fri Dec 13 21:01:26 2013 +0100
> 
>     [release] Removes unused parameter warnings with -Wall -Wextra
> 
>  doc/comm/poosc08_test.cc             | 2 +-
>  dune/common/mallocallocator.hh       | 1 +
>  dune/common/parallel/communicator.hh | 4 ++--
>  dune/common/parallel/mpihelper.hh    | 1 +
>  4 files changed, 5 insertions(+), 3 deletions(-)
> 
> 
> 
> diff --git a/doc/comm/poosc08_test.cc b/doc/comm/poosc08_test.cc
> index 2bef041..2fe7eae 100644
> --- a/doc/comm/poosc08_test.cc
> +++ b/doc/comm/poosc08_test.cc
> @@ -43,7 +43,7 @@ struct CopyData {
>  
>  
>  template<class T>
> -void doCalculations(T& t){}
> +void doCalculations(T&){}
>  
>  #if HAVE_MPI
>  void test()
> diff --git a/dune/common/mallocallocator.hh b/dune/common/mallocallocator.hh
> index daca655..3328a69 100644
> --- a/dune/common/mallocallocator.hh
> +++ b/dune/common/mallocallocator.hh
> @@ -65,6 +65,7 @@ namespace Dune
>      //! deallocate n objects of type T at address p
>      void deallocate(pointer p, size_type n)
>      {
> +      (void)n;
>        std::free(p);
>      }
>  
> diff --git a/dune/common/parallel/communicator.hh b/dune/common/parallel/communicator.hh
> index 69c4fc1..11a7834 100644
> --- a/dune/common/parallel/communicator.hh
> +++ b/dune/common/parallel/communicator.hh
> @@ -1232,7 +1232,7 @@ namespace Dune
>  
>    template<class Data>
>    inline int BufferedCommunicator::MessageSizeCalculator<Data,SizeOne>::operator()
> -    (const Data& data, const InterfaceInformation& info) const
> +    (const Data&, const InterfaceInformation& info) const
>    {
>      return operator()(info);
>    }
> @@ -1285,7 +1285,7 @@ namespace Dune
>  
>  
>    template<class Data, class GatherScatter, bool FORWARD>
> -  inline void BufferedCommunicator::MessageGatherer<Data,GatherScatter,FORWARD,SizeOne>::operator()(const InterfaceMap& interfaces, const Data& data, Type* buffer, size_t bufferSize) const
> +  inline void BufferedCommunicator::MessageGatherer<Data,GatherScatter,FORWARD,SizeOne>::operator()(const InterfaceMap& interfaces, const Data& data, Type* buffer, size_t) const
>    {
>      typedef typename InterfaceMap::const_iterator
>      const_iterator;
> diff --git a/dune/common/parallel/mpihelper.hh b/dune/common/parallel/mpihelper.hh
> index fedc945..01325b2 100644
> --- a/dune/common/parallel/mpihelper.hh
> +++ b/dune/common/parallel/mpihelper.hh
> @@ -129,6 +129,7 @@ namespace Dune
>       */
>      DUNE_EXPORT static FakeMPIHelper& instance(int argc, char** argv)
>      {
> +      (void)argc; (void)argv;
>        // create singleton instance
>        static FakeMPIHelper singleton;
>        return singleton;
> 
> _______________________________________________
> Dune-Commit mailing list
> Dune-Commit at dune-project.org
> http://lists.dune-project.org/mailman/listinfo/dune-commit
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 551 bytes
Desc: OpenPGP digital signature
URL: <https://lists.dune-project.org/pipermail/dune-devel/attachments/20131216/6929f050/attachment.sig>


More information about the Dune-devel mailing list