From 128148a7a2078edce5e8c8a25e0fe69c6561eb28 Mon Sep 17 00:00:00 2001
From: Sebastian Eibl <sebastian.eibl@fau.de>
Date: Wed, 28 Aug 2019 12:38:57 +0200
Subject: [PATCH] [CLANG-TIDY] readability-redundant-smartptr-get

---
 .clang-tidy                                             | 3 ++-
 apps/benchmarks/PoiseuilleChannel/PoiseuilleChannel.cpp | 4 ++--
 apps/benchmarks/SchaeferTurek/SchaeferTurek.cpp         | 4 ++--
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/.clang-tidy b/.clang-tidy
index b32e79a9a..9ac2c70c5 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -38,7 +38,8 @@ readability-delete-null-pointer,
 readability-deleted-default,
 readability-misleading-indentation,
 readability-misplaced-array-index,
-readability-redundant-control-flow
+readability-redundant-control-flow,
+readability-redundant-smartptr-get
 
 '
 WarningsAsErrors: '*'
diff --git a/apps/benchmarks/PoiseuilleChannel/PoiseuilleChannel.cpp b/apps/benchmarks/PoiseuilleChannel/PoiseuilleChannel.cpp
index ae5352c0a..b72efbdcb 100644
--- a/apps/benchmarks/PoiseuilleChannel/PoiseuilleChannel.cpp
+++ b/apps/benchmarks/PoiseuilleChannel/PoiseuilleChannel.cpp
@@ -553,12 +553,12 @@ void setFlags( shared_ptr< StructuredBlockForest > & blocks, const BlockDataID &
       {
          CurvedDeltaValueCalculation< LatticeModel_T > deltaCalculation( blocks, *block, channel );
 
-         lbm::refinement::consistentlyForceBoundary< BoundaryHandling_T >( *blocks, dynamic_cast< blockforest::Block & >(*block.get()),
+         lbm::refinement::consistentlyForceBoundary< BoundaryHandling_T >( *blocks, dynamic_cast< blockforest::Block & >(*block),
                                                                            boundaryHandlingId, Curved_Flag, channel, deltaCalculation );
       }
       else // staircase (1st order bounce back no-slip boundary condition)
       {
-         lbm::refinement::consistentlyForceBoundary< BoundaryHandling_T >( *blocks, dynamic_cast< blockforest::Block & >(*block.get()),
+         lbm::refinement::consistentlyForceBoundary< BoundaryHandling_T >( *blocks, dynamic_cast< blockforest::Block & >(*block),
                                                                            boundaryHandlingId, NoSlip_Flag, channel );
       }
 
diff --git a/apps/benchmarks/SchaeferTurek/SchaeferTurek.cpp b/apps/benchmarks/SchaeferTurek/SchaeferTurek.cpp
index 876a60942..50c3a9244 100644
--- a/apps/benchmarks/SchaeferTurek/SchaeferTurek.cpp
+++ b/apps/benchmarks/SchaeferTurek/SchaeferTurek.cpp
@@ -961,12 +961,12 @@ void BoundarySetter< LatticeModel_T >::operator()()
       {
          CurvedDeltaValueCalculation< LatticeModel_T > deltaCalculation( blocks, *block, cylinder );
 
-         lbm::refinement::consistentlyForceBoundary< BoundaryHandling_T >( *blocks, dynamic_cast< blockforest::Block & >(*block.get()),
+         lbm::refinement::consistentlyForceBoundary< BoundaryHandling_T >( *blocks, dynamic_cast< blockforest::Block & >(*block),
                                                                            boundaryHandlingId_, Curved_Flag, cylinder, deltaCalculation );
       }
       else // staircase
       {
-         lbm::refinement::consistentlyForceBoundary< BoundaryHandling_T >( *blocks, dynamic_cast< blockforest::Block & >(*block.get()),
+         lbm::refinement::consistentlyForceBoundary< BoundaryHandling_T >( *blocks, dynamic_cast< blockforest::Block & >(*block),
                                                                            boundaryHandlingId_, Obstacle_Flag, cylinder );
       }
 
-- 
GitLab