we do need it for RN

Author: mydea

packages/core/src/carrier.ts

@@ -27,8 +27,6 @@ export interface SentryCarrier {
   globalMetricsAggregators?: WeakMap<Client, MetricsAggregator> | undefined;
   logger?: Logger;
 
-  // TODO(v9): Do we still need those?
-  // --> Check with RN
   /** Overwrites TextEncoder used in `@sentry/core`, need for `react-native@0.73` and older */
   encodePolyfill?: (input: string) => Uint8Array;
   /** Overwrites TextDecoder used in `@sentry/core`, need for `react-native@0.73` and older */

packages/core/src/utils-hoist/envelope.ts

@@ -75,8 +75,6 @@ export function envelopeContainsItemType(envelope: Envelope, types: EnvelopeItem
  * Encode a string to UTF8 array.
  */
 function encodeUTF8(input: string): Uint8Array {
-  // TODO(v9): Do we actually still need this?
-  // --> Evaluate if RN still needs this
   const carrier = getSentryCarrier(GLOBAL_OBJ);
   return carrier.encodePolyfill ? carrier.encodePolyfill(input) : new TextEncoder().encode(input);
 }
@@ -85,8 +83,6 @@ function encodeUTF8(input: string): Uint8Array {
  * Decode a UTF8 array to string.
  */
 function decodeUTF8(input: Uint8Array): string {
-  // TODO(v9): Do we actually still need this?
-  // --> Evaluate if RN still needs this
   const carrier = getSentryCarrier(GLOBAL_OBJ);
   return carrier.decodePolyfill ? carrier.decodePolyfill(input) : new TextDecoder().decode(input);
 }