Use curly_braces_in_flow_control_structures for animations, cupertino (#104612)

* Use `curly_braces_in_flow_control_structures` for `animation`

* Use `curly_braces_in_flow_control_structures` for `cupertino`

* fix comments

Author: guidezpl

packages/flutter/lib/src/animation/animation_controller.dart

@@ -389,8 +389,9 @@ class AnimationController extends Animation<double>
   /// If [isAnimating] is false, then [value] is not changing and the rate of
   /// change is zero.
   double get velocity {
-    if (!isAnimating)
+    if (!isAnimating) {
       return 0.0;
+    }
     return _simulation!.dx(lastElapsedDuration!.inMicroseconds.toDouble() / Duration.microsecondsPerSecond);
   }
 
@@ -456,8 +457,9 @@ class AnimationController extends Animation<double>
       'AnimationController methods should not be used after calling dispose.',
     );
     _direction = _AnimationDirection.forward;
-    if (from != null)
+    if (from != null) {
       value = from;
+    }
     return _animateToInternal(upperBound);
   }
 
@@ -489,8 +491,9 @@ class AnimationController extends Animation<double>
       'AnimationController methods should not be used after calling dispose.',
     );
     _direction = _AnimationDirection.reverse;
-    if (from != null)
+    if (from != null) {
       value = from;
+    }
     return _animateToInternal(lowerBound);
   }
 
@@ -856,12 +859,13 @@ class _InterpolationSimulation extends Simulation {
   @override
   double x(double timeInSeconds) {
     final double t = clampDouble(timeInSeconds / _durationInSeconds, 0.0, 1.0);
-    if (t == 0.0)
+    if (t == 0.0) {
       return _begin;
-    else if (t == 1.0)
+    } else if (t == 1.0) {
       return _end;
-    else
+    } else {
       return _begin + (_end - _begin) * _curve.transform(t);
+    }
   }
 
   @override

packages/flutter/lib/src/animation/animations.dart

@@ -188,22 +188,27 @@ class ProxyAnimation extends Animation<double>
   Animation<double>? get parent => _parent;
   Animation<double>? _parent;
   set parent(Animation<double>? value) {
-    if (value == _parent)
+    if (value == _parent) {
       return;
+    }
     if (_parent != null) {
       _status = _parent!.status;
       _value = _parent!.value;
-      if (isListening)
+      if (isListening) {
         didStopListening();
+      }
     }
     _parent = value;
     if (_parent != null) {
-      if (isListening)
+      if (isListening) {
         didStartListening();
-      if (_value != _parent!.value)
+      }
+      if (_value != _parent!.value) {
         notifyListeners();
-      if (_status != _parent!.status)
+      }
+      if (_status != _parent!.status) {
         notifyStatusListeners(_parent!.status);
+      }
       _status = null;
       _value = null;
     }
@@ -233,8 +238,9 @@ class ProxyAnimation extends Animation<double>
 
   @override
   String toString() {
-    if (parent == null)
+    if (parent == null) {
       return '${objectRuntimeType(this, 'ProxyAnimation')}(null; ${super.toStringDetails()} ${value.toStringAsFixed(3)})';
+    }
     return '$parent\u27A9${objectRuntimeType(this, 'ProxyAnimation')}';
   }
 }
@@ -441,8 +447,9 @@ class CurvedAnimation extends Animation<double> with AnimationWithParentMixin<do
     final Curve? activeCurve = _useForwardCurve ? curve : reverseCurve;
 
     final double t = parent.value;
-    if (activeCurve == null)
+    if (activeCurve == null) {
       return t;
+    }
     if (t == 0.0 || t == 1.0) {
       assert(() {
         final double transformedValue = activeCurve.transform(t);
@@ -464,10 +471,12 @@ class CurvedAnimation extends Animation<double> with AnimationWithParentMixin<do
 
   @override
   String toString() {
-    if (reverseCurve == null)
+    if (reverseCurve == null) {
       return '$parent\u27A9$curve';
-    if (_useForwardCurve)
+    }
+    if (_useForwardCurve) {
       return '$parent\u27A9$curve\u2092\u2099/$reverseCurve';
+    }
     return '$parent\u27A9$curve/$reverseCurve\u2092\u2099';
   }
 }
@@ -581,8 +590,9 @@ class TrainHoppingAnimation extends Animation<double>
       _lastValue = newValue;
     }
     assert(_lastValue != null);
-    if (hop && onSwitchedTrain != null)
+    if (hop && onSwitchedTrain != null) {
       onSwitchedTrain!();
+    }
   }
 
   @override
@@ -605,8 +615,9 @@ class TrainHoppingAnimation extends Animation<double>
 
   @override
   String toString() {
-    if (_nextTrain != null)
+    if (_nextTrain != null) {
       return '$currentTrain\u27A9${objectRuntimeType(this, 'TrainHoppingAnimation')}(next: $_nextTrain)';
+    }
     return '$currentTrain\u27A9${objectRuntimeType(this, 'TrainHoppingAnimation')}(no next)';
   }
 }
@@ -660,8 +671,9 @@ abstract class CompoundAnimation<T> extends Animation<T>
   /// Otherwise, default to [first].
   @override
   AnimationStatus get status {
-    if (next.status == AnimationStatus.forward || next.status == AnimationStatus.reverse)
+    if (next.status == AnimationStatus.forward || next.status == AnimationStatus.reverse) {
       return next.status;
+    }
     return first.status;
   }
 

packages/flutter/lib/src/animation/curves.dart

@@ -183,15 +183,17 @@ class Interval extends Curve {
     assert(end <= 1.0);
     assert(end >= begin);
     t = clampDouble((t - begin) / (end - begin), 0.0, 1.0);
-    if (t == 0.0 || t == 1.0)
+    if (t == 0.0 || t == 1.0) {
       return t;
+    }
     return curve.transform(t);
   }
 
   @override
   String toString() {
-    if (curve is! _Linear)
+    if (curve is! _Linear) {
       return '${objectRuntimeType(this, 'Interval')}($begin\u22EF$end)\u27A9$curve';
+    }
     return '${objectRuntimeType(this, 'Interval')}($begin\u22EF$end)';
   }
 }
@@ -348,12 +350,14 @@ class Cubic extends Curve {
     while (true) {
       final double midpoint = (start + end) / 2;
       final double estimate = _evaluateCubic(a, c, midpoint);
-      if ((t - estimate).abs() < _cubicErrorBound)
+      if ((t - estimate).abs() < _cubicErrorBound) {
         return _evaluateCubic(b, d, midpoint);
-      if (estimate < t)
+      }
+      if (estimate < t) {
         start = midpoint;
-      else
+      } else {
         end = midpoint;
+      }
     }
   }
 
@@ -1219,10 +1223,11 @@ class _BounceInOutCurve extends Curve {
 
   @override
   double transformInternal(double t) {
-    if (t < 0.5)
+    if (t < 0.5) {
       return (1.0 - _bounce(1.0 - t * 2.0)) * 0.5;
-    else
+    } else {
       return _bounce(t * 2.0 - 1.0) * 0.5 + 0.5;
+    }
   }
 }
 
@@ -1304,10 +1309,11 @@ class ElasticInOutCurve extends Curve {
   double transformInternal(double t) {
     final double s = period / 4.0;
     t = 2.0 * t - 1.0;
-    if (t < 0.0)
+    if (t < 0.0) {
       return -0.5 * math.pow(2.0, 10.0 * t) * math.sin((t - s) * (math.pi * 2.0) / period);
-    else
+    } else {
       return math.pow(2.0, -10.0 * t) * math.sin((t - s) * (math.pi * 2.0) / period) * 0.5 + 1.0;
+    }
   }
 
   @override

packages/flutter/lib/src/animation/listener_helpers.dart

@@ -26,8 +26,9 @@ mixin AnimationLazyListenerMixin {
   @protected
   void didRegisterListener() {
     assert(_listenerCounter >= 0);
-    if (_listenerCounter == 0)
+    if (_listenerCounter == 0) {
       didStartListening();
+    }
     _listenerCounter += 1;
   }
 
@@ -41,8 +42,9 @@ mixin AnimationLazyListenerMixin {
   void didUnregisterListener() {
     assert(_listenerCounter >= 1);
     _listenerCounter -= 1;
-    if (_listenerCounter == 0)
+    if (_listenerCounter == 0) {
       didStopListening();
+    }
   }
 
   /// Called when the number of listeners changes from zero to one.
@@ -151,8 +153,9 @@ mixin AnimationLocalListenersMixin {
         return true;
       }());
       try {
-        if (_listeners.contains(listener))
+        if (_listeners.contains(listener)) {
           listener();
+        }
       } catch (exception, stack) {
         FlutterError.reportError(FlutterErrorDetails(
           exception: exception,
@@ -229,8 +232,9 @@ mixin AnimationLocalStatusListenersMixin {
     final List<AnimationStatusListener> localListeners = _statusListeners.toList(growable: false);
     for (final AnimationStatusListener listener in localListeners) {
       try {
-        if (_statusListeners.contains(listener))
+        if (_statusListeners.contains(listener)) {
           listener(status);
+        }
       } catch (exception, stack) {
         InformationCollector? collector;
         assert(() {

packages/flutter/lib/src/animation/tween.dart

@@ -320,10 +320,12 @@ class Tween<T extends Object?> extends Animatable<T> {
   /// subclass.
   @override
   T transform(double t) {
-    if (t == 0.0)
+    if (t == 0.0) {
       return begin as T;
-    if (t == 1.0)
+    }
+    if (t == 1.0) {
       return end as T;
+    }
     return lerp(t);
   }
 

packages/flutter/lib/src/animation/tween_sequence.dart

@@ -56,8 +56,9 @@ class TweenSequence<T> extends Animatable<T> {
     _items.addAll(items);
 
     double totalWeight = 0.0;
-    for (final TweenSequenceItem<T> item in _items)
+    for (final TweenSequenceItem<T> item in _items) {
       totalWeight += item.weight;
+    }
     assert(totalWeight > 0.0);
 
     double start = 0.0;
@@ -80,11 +81,13 @@ class TweenSequence<T> extends Animatable<T> {
   @override
   T transform(double t) {
     assert(t >= 0.0 && t <= 1.0);
-    if (t == 1.0)
+    if (t == 1.0) {
       return _evaluateAt(t, _items.length - 1);
+    }
     for (int index = 0; index < _items.length; index++) {
-      if (_intervals[index].contains(t))
+      if (_intervals[index].contains(t)) {
         return _evaluateAt(t, index);
+      }
     }
     // Should be unreachable.
     throw StateError('TweenSequence.evaluate() could not find an interval for $t');

packages/flutter/lib/src/cupertino/activity_indicator.dart

@@ -107,10 +107,11 @@ class _CupertinoActivityIndicatorState extends State<CupertinoActivityIndicator>
   void didUpdateWidget(CupertinoActivityIndicator oldWidget) {
     super.didUpdateWidget(oldWidget);
     if (widget.animating != oldWidget.animating) {
-      if (widget.animating)
+      if (widget.animating) {
         _controller.repeat();
-      else
+      } else {
         _controller.stop();
+      }
     }
   }
 

packages/flutter/lib/src/cupertino/bottom_tab_bar.dart

@@ -271,8 +271,9 @@ class CupertinoTabBar extends StatelessWidget implements PreferredSizeWidget {
 
   /// Change the active tab item's icon and title colors to active.
   Widget _wrapActiveItem(BuildContext context, Widget item, { required bool active }) {
-    if (!active)
+    if (!active) {
       return item;
+    }
 
     final Color activeColor = CupertinoDynamicColor.resolve(
       this.activeColor ?? CupertinoTheme.of(context).primaryColor,

packages/flutter/lib/src/cupertino/button.dart

@@ -214,15 +214,17 @@ class _CupertinoButtonState extends State<CupertinoButton> with SingleTickerProv
   }
 
   void _animate() {
-    if (_animationController.isAnimating)
+    if (_animationController.isAnimating) {
       return;
+    }
     final bool wasHeldDown = _buttonHeldDown;
     final TickerFuture ticker = _buttonHeldDown
         ? _animationController.animateTo(1.0, duration: kFadeOutDuration, curve: Curves.easeInOutCubicEmphasized)
         : _animationController.animateTo(0.0, duration: kFadeInDuration, curve: Curves.easeOutCubic);
     ticker.then<void>((void value) {
-      if (mounted && wasHeldDown != _buttonHeldDown)
+      if (mounted && wasHeldDown != _buttonHeldDown) {
         _animate();
+      }
     });
   }