Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit f467eee

Browse files
authored
Turned on performance-move-const-arg everywhere. (#37482)
Turned on performance-move-const-arg everywhere.
1 parent 057c53a commit f467eee

File tree

4 files changed

+3
-7
lines changed

4 files changed

+3
-7
lines changed

.clang-tidy

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ readability-identifier-naming,\
2929
clang-diagnostic-*,\
3030
google-objc-*,\
3131
google-explicit-constructor,\
32+
performance-move-const-arg,\
3233
performance-unnecessary-value-param"
3334

3435
CheckOptions:

ci/lint.sh

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,6 @@ SRC_DIR="$(cd "$SCRIPT_DIR/../.."; pwd -P)"
3232
FLUTTER_DIR="$(cd "$SCRIPT_DIR/.."; pwd -P)"
3333
DART_BIN="${SRC_DIR}/third_party/dart/tools/sdks/dart-sdk/bin"
3434
DART="${DART_BIN}/dart"
35-
# TODO(https://github.com/flutter/flutter/issues/113848): Migrate all platforms
36-
# to have this as an error.
37-
MAC_HOST_WARNINGS_AS_ERRORS="performance-move-const-arg"
3835

3936
# FLUTTER_LINT_PRINT_FIX will make it so that fix is executed and the generated
4037
# diff is printed to stdout if clang-tidy fails. This is helpful for enabling
@@ -57,7 +54,6 @@ cd "$SCRIPT_DIR"
5754
--disable-dart-dev \
5855
"$SRC_DIR/flutter/tools/clang_tidy/bin/main.dart" \
5956
--src-dir="$SRC_DIR" \
60-
--mac-host-warnings-as-errors="$MAC_HOST_WARNINGS_AS_ERRORS" \
6157
$fix_flag \
6258
"$@" && true # errors ignored
6359
clang_tidy_return=$?

impeller/renderer/backend/vulkan/pipeline_library_vk.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ std::unique_ptr<PipelineCreateInfoVK> PipelineLibraryVK::CreatePipeline(
262262
info.setPName("main");
263263
info.setModule(
264264
ShaderFunctionVK::Cast(entrypoint.second.get())->GetModule());
265-
shader_stages.push_back(std::move(info));
265+
shader_stages.push_back(info);
266266
}
267267
pipeline_info.setStages(shader_stages);
268268

shell/platform/darwin/ios/framework/Source/FlutterPlatformViews.mm

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -734,8 +734,7 @@ - (BOOL)flt_hasFirstResponderInViewHierarchySubtree {
734734
int64_t view_id,
735735
int64_t overlay_id) {
736736
FML_DCHECK(flutter_view_);
737-
std::shared_ptr<FlutterPlatformViewLayer> layer =
738-
layer_pool_->GetLayer(gr_context, std::move(ios_context));
737+
std::shared_ptr<FlutterPlatformViewLayer> layer = layer_pool_->GetLayer(gr_context, ios_context);
739738

740739
UIView* overlay_view_wrapper = layer->overlay_view_wrapper.get();
741740
auto screenScale = [UIScreen mainScreen].scale;

0 commit comments

Comments
 (0)