Remove unnecessary comments.

5 years ago · ad9a9a7067
parent e0a748d2e4
commit ad9a9a7067
2 changed files with 0 additions and 3 deletions
--- a/src/core/lib/security/security_connector/tls/tls_security_connector.h
+++ b/src/core/lib/security/security_connector/tls/tls_security_connector.h
@ -167,7 +167,6 @@ class TlsServerSecurityConnector final : public grpc_server_security_connector {
 *    false denotes that this method is called by a client.
 *  - reload_status: the status of the credential reloading after the method
 *    returns; the caller should not pass in nullptr. **/
-// TODO: is there a memory leak if key materials config is already populated?
 grpc_status_code TlsFetchKeyMaterials(
    const grpc_core::RefCountedPtr<grpc_tls_key_materials_config>&
        key_materials_config,
--- a/test/core/security/tls_security_connector_test.cc
+++ b/test/core/security/tls_security_connector_test.cc
@ -124,8 +124,6 @@ TEST_F(TlsSecurityConnectorTest, NoKeysAndConfig) {
  options_->Unref();
 }

-/** This is the current behavior. Why do we want this to be different for the
- *  client and server? **/
 TEST_F(TlsSecurityConnectorTest, NoKeysAndConfigAsAClient) {
  grpc_ssl_certificate_config_reload_status reload_status;
  grpc_status_code status =