Version 2.12.0-150.0.dev

Merge commit 'ab2645c9f4b863d364658e9f05cc27de42b0204d' into 'dev'

Author: Dart CI

.dart_tool/package_config.json

@@ -18,7 +18,7 @@
       "name": "_fe_analyzer_shared",
       "rootUri": "../pkg/_fe_analyzer_shared",
       "packageUri": "lib/",
-      "languageVersion": "2.6"
+      "languageVersion": "2.12"
     },
     {
       "name": "_fe_analyzer_shared_assigned_variables",
@@ -680,7 +680,7 @@
       "name": "test_reflective_loader",
       "rootUri": "../third_party/pkg/test_reflective_loader",
       "packageUri": "lib/",
-      "languageVersion": "1.8"
+      "languageVersion": "2.12"
     },
     {
       "name": "test_runner",

DEPS

@@ -156,7 +156,7 @@ vars = {
   "test_descriptor_tag": "1.1.1",
   "test_process_tag": "1.0.3",
   "term_glyph_rev": "6a0f9b6fb645ba75e7a00a4e20072678327a0347",
-  "test_reflective_loader_tag": "0.1.9",
+  "test_reflective_loader_rev": "b76ae201ab9c6f3b90643958965e7cc215a38e9b",
   "test_rev": "e37a93bbeae23b215972d1659ac865d71287ff6a",
   "typed_data_tag": "f94fc57b8e8c0e4fe4ff6cfd8290b94af52d3719",
   "usage_tag": "16fbfd90c58f16e016a295a880bc722d2547d2c9",
@@ -440,7 +440,7 @@ deps = {
       Var("dart_git") + "test_process.git" + "@" + Var("test_process_tag"),
   Var("dart_root") + "/third_party/pkg/test_reflective_loader":
       Var("dart_git") + "test_reflective_loader.git" +
-      "@" + Var("test_reflective_loader_tag"),
+      "@" + Var("test_reflective_loader_rev"),
   Var("dart_root") + "/third_party/pkg/typed_data":
       Var("dart_git") + "typed_data.git" + "@" + Var("typed_data_tag"),
   Var("dart_root") + "/third_party/pkg/usage":

build/toolchain/gcc_toolchain.gni

@@ -204,13 +204,7 @@ template("gcc_toolchain") {
         command += " && " + strip_command
       } else if (defined(invoker.llvm_objcopy)) {
         strip = invoker.llvm_objcopy
-        if (defined(invoker.llvm_objcopy_extra_args)) {
-          extra_args = invoker.llvm_objcopy_extra_args
-        } else {
-          extra_args = ""
-        }
-        strip_command =
-            "${strip} --strip-all ${extra_args} $outfile $stripped_outfile"
+        strip_command = "${strip} --strip-all $outfile $stripped_outfile"
         command += " && " + strip_command
       }
       if (defined(invoker.postlink)) {

build/toolchain/linux/BUILD.gn

@@ -52,17 +52,6 @@ gcc_toolchain("clang_arm") {
   ld = cxx
   llvm_objcopy = "${prefix}/llvm-objcopy"
 
-  # When producing ARM builds we drop .ARM.exidx/extab sections. These sections
-  # don't contain any useful information (we don't use exceptions or
-  # unwind C++ frames and most of the dart binary is in fact not covered by
-  # them), however they have been seen to break dynamic linker in older glibc
-  # versions (pre 2.23) because .ARM.exidx ends up being positioned between
-  # .rel.dyn and .rel.plt sections while older versions of dynamic linker
-  # expect these two sections to appear one after another in the ELF file.
-  # See https://github.com/dart-lang/sdk/issues/41644.
-  llvm_objcopy_extra_args =
-      "--remove-section .ARM.exidx --remove-section .ARM.extab"
-
   toolchain_cpu = "arm"
   toolchain_os = "linux"
   is_clang = true

pkg/_fe_analyzer_shared/lib/src/base/errors.dart

@@ -2,8 +2,6 @@
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
-import 'package:meta/meta.dart';
-
 import 'customized_codes.dart';
 
 /// An error code associated with an [AnalysisError].
@@ -23,7 +21,7 @@ abstract class ErrorCode {
 
   final String _message;
 
-  final String _correction;
+  final String? _correction;
 
   /**
    * Return `true` if diagnostics with this code have documentation for them
@@ -43,25 +41,27 @@ abstract class ErrorCode {
    * given [correction] template.
    */
   const ErrorCode({
-    String correction,
+    String? correction,
     this.hasPublishedDocs = false,
     this.isUnresolvedIdentifier: false,
-    @required String message,
-    @required this.name,
-    @required this.uniqueName,
+    required String message,
+    required this.name,
+    required this.uniqueName,
   })  : _correction = correction,
         _message = message,
+        // ignore: unnecessary_null_comparison
         assert(hasPublishedDocs != null),
+        // ignore: unnecessary_null_comparison
         assert(isUnresolvedIdentifier != null);
 
   @Deprecated('Use the default constructor')
   const ErrorCode.temporary2({
-    String correction,
+    String? correction,
     bool hasPublishedDocs = false,
     bool isUnresolvedIdentifier = false,
-    @required String message,
-    @required String name,
-    @required String uniqueName,
+    required String message,
+    required String name,
+    required String uniqueName,
   }) : this(
           correction: correction,
           hasPublishedDocs: hasPublishedDocs,
@@ -76,7 +76,7 @@ abstract class ErrorCode {
    * or `null` if there is no correction information for this error. The
    * correction should indicate how the user can fix the error.
    */
-  String get correction => customizedCorrections[uniqueName] ?? _correction;
+  String? get correction => customizedCorrections[uniqueName] ?? _correction;
 
   /**
    * The severity of the error.
@@ -102,7 +102,7 @@ abstract class ErrorCode {
    * Return a URL that can be used to access documentation for diagnostics with
    * this code, or `null` if there is no published documentation.
    */
-  String get url {
+  String? get url {
     if (hasPublishedDocs) {
       return 'https://dart.dev/tools/diagnostic-messages#${name.toLowerCase()}';
     }