diff --git a/.clang-tidy b/.clang-tidy index 6bd27668fd974..7947394ce3950 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -29,6 +29,7 @@ readability-identifier-naming,\ clang-diagnostic-*,\ google-objc-*,\ google-explicit-constructor,\ +performance-move-const-arg,\ performance-unnecessary-value-param" CheckOptions: diff --git a/ci/lint.sh b/ci/lint.sh index 7c6c4e0c314b8..ed7ced728409b 100755 --- a/ci/lint.sh +++ b/ci/lint.sh @@ -32,9 +32,6 @@ SRC_DIR="$(cd "$SCRIPT_DIR/../.."; pwd -P)" FLUTTER_DIR="$(cd "$SCRIPT_DIR/.."; pwd -P)" DART_BIN="${SRC_DIR}/third_party/dart/tools/sdks/dart-sdk/bin" DART="${DART_BIN}/dart" -# TODO(https://github.com/flutter/flutter/issues/113848): Migrate all platforms -# to have this as an error. -MAC_HOST_WARNINGS_AS_ERRORS="performance-move-const-arg" # FLUTTER_LINT_PRINT_FIX will make it so that fix is executed and the generated # diff is printed to stdout if clang-tidy fails. This is helpful for enabling @@ -57,7 +54,6 @@ cd "$SCRIPT_DIR" --disable-dart-dev \ "$SRC_DIR/flutter/tools/clang_tidy/bin/main.dart" \ --src-dir="$SRC_DIR" \ - --mac-host-warnings-as-errors="$MAC_HOST_WARNINGS_AS_ERRORS" \ $fix_flag \ "$@" && true # errors ignored clang_tidy_return=$? diff --git a/impeller/renderer/backend/vulkan/pipeline_library_vk.cc b/impeller/renderer/backend/vulkan/pipeline_library_vk.cc index b282b4fdd9d38..7c6a9e85634f1 100644 --- a/impeller/renderer/backend/vulkan/pipeline_library_vk.cc +++ b/impeller/renderer/backend/vulkan/pipeline_library_vk.cc @@ -262,7 +262,7 @@ std::unique_ptr PipelineLibraryVK::CreatePipeline( info.setPName("main"); info.setModule( ShaderFunctionVK::Cast(entrypoint.second.get())->GetModule()); - shader_stages.push_back(std::move(info)); + shader_stages.push_back(info); } pipeline_info.setStages(shader_stages); diff --git a/shell/platform/darwin/ios/framework/Source/FlutterPlatformViews.mm b/shell/platform/darwin/ios/framework/Source/FlutterPlatformViews.mm index 90885780576b0..51085c935577f 100644 --- a/shell/platform/darwin/ios/framework/Source/FlutterPlatformViews.mm +++ b/shell/platform/darwin/ios/framework/Source/FlutterPlatformViews.mm @@ -734,8 +734,7 @@ - (BOOL)flt_hasFirstResponderInViewHierarchySubtree { int64_t view_id, int64_t overlay_id) { FML_DCHECK(flutter_view_); - std::shared_ptr layer = - layer_pool_->GetLayer(gr_context, std::move(ios_context)); + std::shared_ptr layer = layer_pool_->GetLayer(gr_context, ios_context); UIView* overlay_view_wrapper = layer->overlay_view_wrapper.get(); auto screenScale = [UIScreen mainScreen].scale;