diff --git a/packages/camera/camera/example/lib/main.dart b/packages/camera/camera/example/lib/main.dart index 120e7f6ec50..d048fc794f2 100644 --- a/packages/camera/camera/example/lib/main.dart +++ b/packages/camera/camera/example/lib/main.dart @@ -534,7 +534,7 @@ class _CameraExampleHomeState extends State onPressed: cameraController != null && cameraController.value.isInitialized && cameraController.value.isRecordingVideo - ? (cameraController.value.isRecordingPaused) + ? cameraController.value.isRecordingPaused ? onResumeButtonPressed : onPauseButtonPressed : null, diff --git a/packages/camera/camera_android/example/lib/main.dart b/packages/camera/camera_android/example/lib/main.dart index 0b587ff669a..47c40c7d20d 100644 --- a/packages/camera/camera_android/example/lib/main.dart +++ b/packages/camera/camera_android/example/lib/main.dart @@ -540,7 +540,7 @@ class _CameraExampleHomeState extends State onPressed: cameraController != null && cameraController.value.isInitialized && cameraController.value.isRecordingVideo - ? (cameraController.value.isRecordingPaused) + ? cameraController.value.isRecordingPaused ? onResumeButtonPressed : onPauseButtonPressed : null, diff --git a/packages/camera/camera_avfoundation/example/lib/main.dart b/packages/camera/camera_avfoundation/example/lib/main.dart index 6bbfede262c..f0ebf7b7865 100644 --- a/packages/camera/camera_avfoundation/example/lib/main.dart +++ b/packages/camera/camera_avfoundation/example/lib/main.dart @@ -540,7 +540,7 @@ class _CameraExampleHomeState extends State onPressed: cameraController != null && cameraController.value.isInitialized && cameraController.value.isRecordingVideo - ? (cameraController.value.isRecordingPaused) + ? cameraController.value.isRecordingPaused ? onResumeButtonPressed : onPauseButtonPressed : null, diff --git a/packages/google_maps_flutter/google_maps_flutter/example/lib/place_polygon.dart b/packages/google_maps_flutter/google_maps_flutter/example/lib/place_polygon.dart index e18d088ed31..11b3ed6e0bb 100644 --- a/packages/google_maps_flutter/google_maps_flutter/example/lib/place_polygon.dart +++ b/packages/google_maps_flutter/google_maps_flutter/example/lib/place_polygon.dart @@ -225,7 +225,7 @@ class PlacePolygonBodyState extends State { TextButton( onPressed: (selectedId == null) ? null - : ((polygons[selectedId]!.holes.isNotEmpty) + : (polygons[selectedId]!.holes.isNotEmpty ? null : () => _addHoles(selectedId)), child: const Text('add holes'), @@ -233,7 +233,7 @@ class PlacePolygonBodyState extends State { TextButton( onPressed: (selectedId == null) ? null - : ((polygons[selectedId]!.holes.isEmpty) + : (polygons[selectedId]!.holes.isEmpty ? null : () => _removeHoles(selectedId)), child: const Text('remove holes'), diff --git a/packages/google_maps_flutter/google_maps_flutter_android/example/lib/place_polygon.dart b/packages/google_maps_flutter/google_maps_flutter_android/example/lib/place_polygon.dart index 889c38d6376..a1fdafd7a31 100644 --- a/packages/google_maps_flutter/google_maps_flutter_android/example/lib/place_polygon.dart +++ b/packages/google_maps_flutter/google_maps_flutter_android/example/lib/place_polygon.dart @@ -226,7 +226,7 @@ class PlacePolygonBodyState extends State { TextButton( onPressed: (selectedId == null) ? null - : ((polygons[selectedId]!.holes.isNotEmpty) + : (polygons[selectedId]!.holes.isNotEmpty ? null : () => _addHoles(selectedId)), child: const Text('add holes'), @@ -234,7 +234,7 @@ class PlacePolygonBodyState extends State { TextButton( onPressed: (selectedId == null) ? null - : ((polygons[selectedId]!.holes.isEmpty) + : (polygons[selectedId]!.holes.isEmpty ? null : () => _removeHoles(selectedId)), child: const Text('remove holes'), diff --git a/packages/google_maps_flutter/google_maps_flutter_ios/example/shared/maps_example_dart/lib/place_polygon.dart b/packages/google_maps_flutter/google_maps_flutter_ios/example/shared/maps_example_dart/lib/place_polygon.dart index 889c38d6376..a1fdafd7a31 100644 --- a/packages/google_maps_flutter/google_maps_flutter_ios/example/shared/maps_example_dart/lib/place_polygon.dart +++ b/packages/google_maps_flutter/google_maps_flutter_ios/example/shared/maps_example_dart/lib/place_polygon.dart @@ -226,7 +226,7 @@ class PlacePolygonBodyState extends State { TextButton( onPressed: (selectedId == null) ? null - : ((polygons[selectedId]!.holes.isNotEmpty) + : (polygons[selectedId]!.holes.isNotEmpty ? null : () => _addHoles(selectedId)), child: const Text('add holes'), @@ -234,7 +234,7 @@ class PlacePolygonBodyState extends State { TextButton( onPressed: (selectedId == null) ? null - : ((polygons[selectedId]!.holes.isEmpty) + : (polygons[selectedId]!.holes.isEmpty ? null : () => _removeHoles(selectedId)), child: const Text('remove holes'), diff --git a/packages/pigeon/CHANGELOG.md b/packages/pigeon/CHANGELOG.md index ec8daa80804..dea1af96901 100644 --- a/packages/pigeon/CHANGELOG.md +++ b/packages/pigeon/CHANGELOG.md @@ -218,8 +218,6 @@ `YES`. Any data class or Flutter API interactions involving `bool`s should be carefully audited by hand when updating. - - ## 12.0.1 * [swift] Adds protocol for Flutter APIs. diff --git a/packages/pigeon/lib/dart_generator.dart b/packages/pigeon/lib/dart_generator.dart index b6d81bbdd77..e8c7550cd70 100644 --- a/packages/pigeon/lib/dart_generator.dart +++ b/packages/pigeon/lib/dart_generator.dart @@ -2168,11 +2168,11 @@ String _addGenericTypes(TypeDeclaration type) { final List typeArguments = type.typeArguments; switch (type.baseName) { case 'List': - return (typeArguments.isEmpty) + return typeArguments.isEmpty ? 'List' : 'List<${_flattenTypeArguments(typeArguments)}>'; case 'Map': - return (typeArguments.isEmpty) + return typeArguments.isEmpty ? 'Map' : 'Map<${_flattenTypeArguments(typeArguments)}>'; default: diff --git a/packages/pigeon/lib/objc_generator.dart b/packages/pigeon/lib/objc_generator.dart index 3a4d8962435..d0e0e99ee29 100644 --- a/packages/pigeon/lib/objc_generator.dart +++ b/packages/pigeon/lib/objc_generator.dart @@ -1515,7 +1515,7 @@ String _propertyTypeForDartType(TypeDeclaration type, } String _capitalize(String str) => - (str.isEmpty) ? '' : str[0].toUpperCase() + str.substring(1); + str.isEmpty ? '' : str[0].toUpperCase() + str.substring(1); /// Returns the components of the objc selector that will be generated from /// [func], ie the strings between the semicolons. [lastSelectorComponent] is diff --git a/packages/webview_flutter/webview_flutter/example/integration_test/webview_flutter_test.dart b/packages/webview_flutter/webview_flutter/example/integration_test/webview_flutter_test.dart index eaf9d7ea309..e9d38e420f4 100644 --- a/packages/webview_flutter/webview_flutter/example/integration_test/webview_flutter_test.dart +++ b/packages/webview_flutter/webview_flutter/example/integration_test/webview_flutter_test.dart @@ -584,7 +584,7 @@ Future main() async { await controller.setNavigationDelegate(NavigationDelegate( onPageFinished: (_) => pageLoaded.complete(), onNavigationRequest: (NavigationRequest navigationRequest) { - return (navigationRequest.url.contains('youtube.com')) + return navigationRequest.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }, @@ -654,7 +654,7 @@ Future main() async { await controller.setNavigationDelegate(NavigationDelegate( onPageFinished: (_) => pageLoaded.complete(), onNavigationRequest: (NavigationRequest navigationRequest) { - return (navigationRequest.url.contains('youtube.com')) + return navigationRequest.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; })); diff --git a/packages/webview_flutter/webview_flutter/example/integration_test/webview_flutter_test_legacy.dart b/packages/webview_flutter/webview_flutter/example/integration_test/webview_flutter_test_legacy.dart index fb037cffd36..076c103bc2f 100644 --- a/packages/webview_flutter/webview_flutter/example/integration_test/webview_flutter_test_legacy.dart +++ b/packages/webview_flutter/webview_flutter/example/integration_test/webview_flutter_test_legacy.dart @@ -873,7 +873,7 @@ Future main() async { }, javascriptMode: JavascriptMode.unrestricted, navigationDelegate: (NavigationRequest request) { - return (request.url.contains('youtube.com')) + return request.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }, @@ -1003,7 +1003,7 @@ Future main() async { }, javascriptMode: JavascriptMode.unrestricted, navigationDelegate: (NavigationRequest request) { - return (request.url.contains('youtube.com')) + return request.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }, diff --git a/packages/webview_flutter/webview_flutter_android/example/integration_test/webview_flutter_test.dart b/packages/webview_flutter/webview_flutter_android/example/integration_test/webview_flutter_test.dart index 4c22df1ff28..d5ed471afd3 100644 --- a/packages/webview_flutter/webview_flutter_android/example/integration_test/webview_flutter_test.dart +++ b/packages/webview_flutter/webview_flutter_android/example/integration_test/webview_flutter_test.dart @@ -867,7 +867,7 @@ Future main() async { await delegate.setOnPageFinished((_) => pageLoaded.complete()); await delegate .setOnNavigationRequest((NavigationRequest navigationRequest) { - return (navigationRequest.url.contains('youtube.com')) + return navigationRequest.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }); @@ -1059,7 +1059,7 @@ Future main() async { await delegate.setOnPageFinished((_) => pageLoaded.complete()); await delegate .setOnNavigationRequest((NavigationRequest navigationRequest) { - return (navigationRequest.url.contains('youtube.com')) + return navigationRequest.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }); diff --git a/packages/webview_flutter/webview_flutter_android/example/integration_test/webview_flutter_test_legacy.dart b/packages/webview_flutter/webview_flutter_android/example/integration_test/webview_flutter_test_legacy.dart index 2e7f3d0b8dd..51f71d7801b 100644 --- a/packages/webview_flutter/webview_flutter_android/example/integration_test/webview_flutter_test_legacy.dart +++ b/packages/webview_flutter/webview_flutter_android/example/integration_test/webview_flutter_test_legacy.dart @@ -1126,7 +1126,7 @@ Future main() async { }, javascriptMode: JavascriptMode.unrestricted, navigationDelegate: (NavigationRequest request) { - return (request.url.contains('youtube.com')) + return request.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }, @@ -1251,7 +1251,7 @@ Future main() async { }, javascriptMode: JavascriptMode.unrestricted, navigationDelegate: (NavigationRequest request) { - return (request.url.contains('youtube.com')) + return request.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }, diff --git a/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/legacy/webview_flutter_test.dart b/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/legacy/webview_flutter_test.dart index f6bf750071d..10d6514582b 100644 --- a/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/legacy/webview_flutter_test.dart +++ b/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/legacy/webview_flutter_test.dart @@ -917,7 +917,7 @@ Future main() async { }, javascriptMode: JavascriptMode.unrestricted, navigationDelegate: (NavigationRequest request) { - return (request.url.contains('youtube.com')) + return request.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }, @@ -1047,7 +1047,7 @@ Future main() async { }, javascriptMode: JavascriptMode.unrestricted, navigationDelegate: (NavigationRequest request) { - return (request.url.contains('youtube.com')) + return request.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }, diff --git a/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/webview_flutter_test.dart b/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/webview_flutter_test.dart index 6f9256687df..84bb246580f 100644 --- a/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/webview_flutter_test.dart +++ b/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/webview_flutter_test.dart @@ -829,7 +829,7 @@ Future main() async { unawaited(delegate.setOnPageFinished((_) => pageLoaded.complete())); unawaited( delegate.setOnNavigationRequest((NavigationRequest navigationRequest) { - return (navigationRequest.url.contains('youtube.com')) + return navigationRequest.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }), @@ -1086,7 +1086,7 @@ Future main() async { unawaited(delegate.setOnPageFinished((_) => pageLoaded.complete())); unawaited(delegate .setOnNavigationRequest((NavigationRequest navigationRequest) { - return (navigationRequest.url.contains('youtube.com')) + return navigationRequest.url.contains('youtube.com') ? NavigationDecision.prevent : NavigationDecision.navigate; }));