*Breaking change* Deprecate CurvedAnimation.reverseCurve in favor of AsymmetricCurvedAnimation

sites/docs/src/content/release/breaking-changes/deprecate-curved-animation-reverse-curve.md

@@ -0,0 +1,126 @@
+---
+title: Deprecate `CurvedAnimation.reverseCurve` in favor of `AsymmetricCurvedAnimation`
+description: >-
+  CurvedAnimation is becoming a single-curve animation.
+  Use AsymmetricCurvedAnimation for different curves in the forward and
+  reverse directions.
+---
+
+{% render "docs/breaking-changes.md" %}
+
+## Summary
+
+[`CurvedAnimation`][]'s [`reverseCurve`][] field has been deprecated.
+
+If needed,
+switch from [`CurvedAnimation`][] to [`AsymmetricCurvedAnimation`][].
+
+## Background
+
+To support its [`reverseCurve`][] functionality,
+`CurvedAnimation` had to add listeners to its [`parent`][]
+to keep track of its direction.
+If you forget to call the [`dispose`][] method when you're done,
+those listeners would stick around in memory and prevent Dart from freeing up
+unneeded resources.
+
+But the most common use case of `CurvedAnimation` is with a single curve,
+meaning these listeners aren't even needed most of the time.
+
+For this reason, [`reverseCurve`][] is being removed from [`CurvedAnimation`][]
+and moved to [`AsymmetricCurvedAnimation`][].
+
+After a migration period, it will be removed from `CurvedAnimation` alongside
+its listeners. This means more memory safety with [`CurvedAnimation`][]
+and less code to write, since you won't need to dispose it.
+
+(Until the migration period is over, remember to still call [`dispose`] when
+you're done.
+If you use [`AsymmetricCurvedAnimation`][], you still need to call its
+[`dispose`][] method regardless.)
+
+## Migration guide
+
+If you need [`CurvedAnimation`][]'s [`reverseCurve`][] field,
+switch from [`CurvedAnimation`][] to [`AsymmetricCurvedAnimation`][].
+
+Code before migration:
+
+```dart
+// This doesn't use `reverseCurve` so it stays as `CurvedAnimation`.
+final oneCurve = CurvedAnimation(
+  parent: _animationController,
+  curve: Curves.easeIn,
+);
+
+// This uses `reverseCurve` so migrate to `AsymmetricCurvedAnimation`.
+final twoCurves = CurvedAnimation(
+  parent: _animationController,
+  curve: Curves.easeIn,
+  reverseCurve: Curves.easeOut,
+);
+```
+
+Code after migration:
+
+```dart
+// This doesn't use `reverseCurve` so it stays as `CurvedAnimation`.
+final oneCurve = CurvedAnimation(
+  parent: _animationController,
+  curve: Curves.easeIn,
+);
+
+// This uses `reverseCurve` so migrate to `AsymmetricCurvedAnimation`.
+final twoCurves = AsymmetricCurvedAnimation(
+  parent: _animationController,
+  curve: Curves.easeIn,
+  reverseCurve: Curves.easeOut,
+);
+```
+
+## Timeline
+
+{% comment %}
+  The version # of the SDK where this change was
+  introduced.  If there is a deprecation window,
+  the version # to which we guarantee to maintain
+  the old API. Use the following template:
+
+  If a breaking change has been reverted in a
+  subsequent release, move that item to the
+  "Reverted" section of the index.md file.
+  Also add the "Reverted in version" line,
+  shown as optional below. Otherwise, delete
+  that line.
+{% endcomment %}
+
+Landed in version: Not yet<br>
+In stable release: Not yet
+
+## References
+
+{% render "docs/main-api.md", site: site %}
+
+API documentation:
+
+* [`CurvedAnimation`][]
+* [`AsymmetricCurvedAnimation`][]
+
+Relevant issues:
+
+* [Disambiguate CurvedAnimation and CurveTween][]
+* [Docs should instruct user to dispose `CurvedAnimation`][]
+
+Relevant PRs:
+
+* [Deprecate `CurvedAnimation.reverseCurve` for `AsymmetricCurvedAnimation`][]
+
+[`AsymmetricCurvedAnimation`]: {{site.main-api}}/flutter/animation/AsymmetricCurvedAnimation-class.html
+[`CurvedAnimation`]: {{site.main-api}}/flutter/animation/CurvedAnimation-class.html
+[`dispose`]: {{site.main-api}}/flutter/animation/AsymmetricCurvedAnimation/dispose.html
+[`parent`]: {{site.main-api}}/flutter/animation/CurvedAnimation/parent.html
+[`reverseCurve`]: {{site.main-api}}/flutter/animation/AsymmetricCurvedAnimation/reverseCurve.html
+
+[Disambiguate CurvedAnimation and CurveTween]: {{site.repo.flutter}}/issues/185468
+[Docs should instruct user to dispose `CurvedAnimation`]: {{site.repo.flutter}}/issues/183292
+[Deprecate `CurvedAnimation.reverseCurve` for `AsymmetricCurvedAnimation`]: {{site.repo.flutter}}/pull/185797

sites/docs/src/content/release/breaking-changes/index.md

@@ -39,6 +39,7 @@ They're sorted by release and listed in alphabetical order:
 * [Changing RawMenuAnchor close order][]
 * [Deprecate `onReorder` callback][]
 * [Deprecated `cacheExtent` and `cacheExtentStyle`][]
+* [Deprecate `CurvedAnimation.reverseCurve` in favor of `AsymmetricCurvedAnimation`][]
 * [Deprecate `TextInputConnection.setStyle`][]
 * [`IconData` class marked as `final`][]
 * [Large screen orientation and resizability restrictions ignored on Android 17][]
@@ -50,6 +51,7 @@ They're sorted by release and listed in alphabetical order:
 [Changing RawMenuAnchor close order]: /release/breaking-changes/raw-menu-anchor-close-order
 [Deprecate `onReorder` callback]: /release/breaking-changes/deprecate-onreorder-callback
 [Deprecated `cacheExtent` and `cacheExtentStyle`]: /release/breaking-changes/scroll-cache-extent
+[Deprecate `CurvedAnimation.reverseCurve` in favor of `AsymmetricCurvedAnimation`]: /release/breaking-changes/deprecate-curved-animation-reverse-curve
 [Deprecate `TextInputConnection.setStyle`]: /release/breaking-changes/deprecate-text-input-connection-set-style
 [`IconData` class marked as `final`]: /release/breaking-changes/icondata-class-marked-final
 [ListTile reports error in debug when wrapped in a colored widget]: /release/breaking-changes/list-tile-color-warning