Skip to content

Commit 80cee59

Browse files
Don't rethrow Dart plugin registration exceptions (flutter#122111)
Don't rethrow Dart plugin registration exceptions
1 parent c0ddbf1 commit 80cee59

File tree

3 files changed

+0
-9
lines changed

3 files changed

+0
-9
lines changed

packages/flutter_tools/lib/src/flutter_plugins.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -703,7 +703,6 @@ const String _dartPluginRegisterWith = r'''
703703
'`{{pluginName}}` threw an error: $err. '
704704
'The app may not function as expected until you remove this plugin from pubspec.yaml'
705705
);
706-
rethrow;
707706
}
708707
''';
709708

packages/flutter_tools/test/general.shard/build_system/targets/dart_plugin_registrant_test.dart

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,7 +257,6 @@ void main() {
257257
" '`path_provider_linux` threw an error: \$err. '\n"
258258
" 'The app may not function as expected until you remove this plugin from pubspec.yaml'\n"
259259
' );\n'
260-
' rethrow;\n'
261260
' }\n'
262261
'\n'
263262
' } else if (Platform.isMacOS) {\n'
@@ -385,7 +384,6 @@ void main() {
385384
" '`path_provider_linux` threw an error: \$err. '\n"
386385
" 'The app may not function as expected until you remove this plugin from pubspec.yaml'\n"
387386
' );\n'
388-
' rethrow;\n'
389387
' }\n'
390388
'\n'
391389
' } else if (Platform.isMacOS) {\n'

packages/flutter_tools/test/general.shard/dart_plugin_test.dart

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -807,7 +807,6 @@ void main() {
807807
" '`url_launcher_android` threw an error: \$err. '\n"
808808
" 'The app may not function as expected until you remove this plugin from pubspec.yaml'\n"
809809
' );\n'
810-
' rethrow;\n'
811810
' }\n'
812811
'\n'
813812
' } else if (Platform.isIOS) {\n'
@@ -818,7 +817,6 @@ void main() {
818817
" '`url_launcher_ios` threw an error: \$err. '\n"
819818
" 'The app may not function as expected until you remove this plugin from pubspec.yaml'\n"
820819
' );\n'
821-
' rethrow;\n'
822820
' }\n'
823821
'\n'
824822
' } else if (Platform.isLinux) {\n'
@@ -829,7 +827,6 @@ void main() {
829827
" '`url_launcher_linux` threw an error: \$err. '\n"
830828
" 'The app may not function as expected until you remove this plugin from pubspec.yaml'\n"
831829
' );\n'
832-
' rethrow;\n'
833830
' }\n'
834831
'\n'
835832
' } else if (Platform.isMacOS) {\n'
@@ -840,7 +837,6 @@ void main() {
840837
" '`awesome_macos` threw an error: \$err. '\n"
841838
" 'The app may not function as expected until you remove this plugin from pubspec.yaml'\n"
842839
' );\n'
843-
' rethrow;\n'
844840
' }\n'
845841
'\n'
846842
' try {\n'
@@ -850,7 +846,6 @@ void main() {
850846
" '`url_launcher_macos` threw an error: \$err. '\n"
851847
" 'The app may not function as expected until you remove this plugin from pubspec.yaml'\n"
852848
' );\n'
853-
' rethrow;\n'
854849
' }\n'
855850
'\n'
856851
' } else if (Platform.isWindows) {\n'
@@ -861,7 +856,6 @@ void main() {
861856
" '`url_launcher_windows` threw an error: \$err. '\n"
862857
" 'The app may not function as expected until you remove this plugin from pubspec.yaml'\n"
863858
' );\n'
864-
' rethrow;\n'
865859
' }\n'
866860
'\n'
867861
' }\n'

0 commit comments

Comments
 (0)