chore: Address or remove all TODOs assigned to themarwhal

Signed-off-by: GitHub <noreply@github.com>

lte/gateway/c/session_manager/ChargingGrant.cpp

@@ -89,9 +89,8 @@ CreditValidity ChargingGrant::get_credit_response_validity_type(
       !gsu.total().is_valid() && !gsu.rx().is_valid() && !gsu.tx().is_valid();
   if (gsu_all_invalid) {
     if (update.credit().is_final() || credit_validity == TRANSIENT_ERROR) {
-      // TODO(@themarwhal): look into this case. Before I figure it out, I will
-      // allow empty GSU credits with FUA or to be suspended
-      // to be on the conservative side.
+      // We will allow empty GSU credits with FUA or to be suspended
+      // to be on the conservative side
       MLOG(MWARNING)
           << "GSU for RG: " << key << " " << session_id
           << " is invalid, but accepting it as it has a final unit action or "

lte/gateway/c/session_manager/SessionState.cpp

@@ -44,8 +44,6 @@ const char* LABEL_SESSION_ID = "session_id";
 const char* LABEL_DIRECTION = "direction";
 const char* DIRECTION_UP = "up";
 const char* DIRECTION_DOWN = "down";
-// TODO(@themarwhal): SessionD should own the naming of the drop all rule so
-// that we never regress here
 const char* DROP_ALL_RULE = "internal_default_drop_flow_rule";
 }  // namespace
 

lte/gateway/c/session_manager/SessionState.hpp

@@ -558,7 +558,7 @@ class SessionState {
   bool add_to_monitor(const std::string& key, uint64_t used_tx,
                       uint64_t used_rx, SessionStateUpdateCriteria* session_uc);
 
-  // TODO(@themarwhal) clean up this function as it is used for testing only
+  // TODO #12593 clean up this function as it is used for testing only
   void set_monitor(const std::string& key, Monitor monitor,
                    SessionStateUpdateCriteria* session_uc);