(AppState) remove outdated TODO (#34264)

Antoine Doubovetzky · facebook-github-bot · commit 0d068a569998 · 2022-07-26T04:12:40.000-07:00
Summary: We are already returning the `remove` function, and the `removeEventListener` method has been removed in this PR: #33580. So I believe this comment is now outdated and no longer relevant. ## Changelog  [Internal] [Removed] - Remove outdated TODO in AppState Pull Request resolved: #34264 Reviewed By: dmitryrykun Differential Revision: D38120344 Pulled By: GijsWeterings fbshipit-source-id: c6c4d42f5b631a67bb6ad49c14df1a772f1cc71b
diff --git a/Libraries/AppState/AppState.js b/Libraries/AppState/AppState.js
@@ -82,11 +82,6 @@ class AppState {
     }
   }
 
-  // TODO: now that AppState is a subclass of NativeEventEmitter, we could
-  // deprecate `addEventListener` and `removeEventListener` and just use
-  // addListener` and `listener.remove()` directly. That will be a breaking
-  // change though, as both the method and event names are different
-  // (addListener events are currently required to be globally unique).
   /**
    * Add a handler to AppState changes by listening to the `change` event type
    * and providing the handler.

Original file line number	Diff line number	Diff line change
`@@ -82,11 +82,6 @@ class AppState {`
`82`	`82`	`}`
`83`	`83`	`}`
`84`	`84`
`85`		`- // TODO: now that AppState is a subclass of NativeEventEmitter, we could`
`86`		- // deprecate `addEventListener` and `removeEventListener` and just use
`87`		- // addListener` and `listener.remove()` directly. That will be a breaking
`88`		`- // change though, as both the method and event names are different`
`89`		`- // (addListener events are currently required to be globally unique).`
`90`	`85`	`/**`
`91`	`86`	* Add a handler to AppState changes by listening to the `change` event type
`92`	`87`	`* and providing the handler.`