Skip to content

Commit

Permalink
Add -Wno-error=redundant-move to cmake builds (#13582)
Browse files Browse the repository at this point in the history
  • Loading branch information
dconeybe authored Sep 4, 2024
1 parent b0dbeb8 commit e0158c0
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions cmake/compiler_setup.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

include(CheckCXXCompilerFlag)

# C++ Compiler setup

# We use C++14
Expand Down Expand Up @@ -76,6 +78,14 @@ if(CXX_CLANG OR CXX_GNU)
list(APPEND common_flags -fdiagnostics-color)
endif()
endif()

# Disable treating "redundant-move" as an error since it's not really a problem,
# and is even a valid coding style to over-use std::move() in case the type is
# ever changed to become non-trivially moveable.
CHECK_CXX_COMPILER_FLAG("-Wno-error=redundant-move" FIREBASE_CXX_COMPILER_FLAG_REDUNDANT_MOVE_SUPPORTED)
if(FIREBASE_CXX_COMPILER_FLAG_REDUNDANT_MOVE_SUPPORTED)
list(APPEND common_flags -Wno-error=redundant-move)
endif()
endif()

if(APPLE)
Expand Down

0 comments on commit e0158c0

Please sign in to comment.