trim comments

Author: typotter

features/dd-sdk-android-flags/src/main/kotlin/com/datadog/android/flags/FlagsClient.kt

@@ -432,7 +432,6 @@ interface FlagsClient {
 
                 val precomputeMapper = PrecomputeMapper(featureSdkCore.internalLogger)
 
-                // Create shared channel for state change notifications
                 val flagStateChannel = FlagsStateChannel(
                     subscription = DDCoreSubscription.create()
                 )

features/dd-sdk-android-flags/src/main/kotlin/com/datadog/android/flags/internal/FlagsStateChannel.kt

@@ -63,20 +63,10 @@ internal class FlagsStateChannel(
         notifyState(FlagsClientState.ERROR, error)
     }
 
-    /**
-     * Registers a listener to receive state change notifications.
-     *
-     * @param listener the listener to register
-     */
     fun addListener(listener: FlagsStateListener) {
         subscription.addListener(listener)
     }
 
-    /**
-     * Unregisters a previously registered listener.
-     *
-     * @param listener the listener to unregister
-     */
     fun removeListener(listener: FlagsStateListener) {
         subscription.removeListener(listener)
     }

features/dd-sdk-android-flags/src/main/kotlin/com/datadog/android/flags/internal/NoOpFlagsClient.kt

@@ -127,15 +127,13 @@ internal class NoOpFlagsClient(
      * @param listener Ignored listener.
      */
     override fun addStateListener(listener: FlagsStateListener) {
-        // No-op: listeners are not supported on NoOpFlagsClient
     }
 
     /**
      * No-op implementation that ignores listener removal.
      * @param listener Ignored listener.
      */
     override fun removeStateListener(listener: FlagsStateListener) {
-        // No-op: listeners are not supported on NoOpFlagsClient
     }
 
     /**

features/dd-sdk-android-flags/src/main/kotlin/com/datadog/android/flags/internal/evaluation/EvaluationsManager.kt

@@ -51,7 +51,6 @@ internal class EvaluationsManager(
      * a valid targeting key.
      */
     fun updateEvaluationsForContext(context: EvaluationContext) {
-        // Transition to RECONCILING before starting the fetch operation
         flagStateChannel.notifyReconciling()
 
         executorService.executeSafe(
@@ -83,7 +82,6 @@ internal class EvaluationsManager(
                 { "Successfully processed context ${context.targetingKey} with ${flagsMap.size} flags" }
             )
 
-            // Transition to READY after successful storage, or ERROR if fetch failed
             if (response != null) {
                 flagStateChannel.notifyReady()
             } else {