diff --git a/src/objective-c/GRPCClient/private/GRPCSecureChannel.m b/src/objective-c/GRPCClient/private/GRPCSecureChannel.m index d3c4d41a139..9b4b6768f84 100644 --- a/src/objective-c/GRPCClient/private/GRPCSecureChannel.m +++ b/src/objective-c/GRPCClient/private/GRPCSecureChannel.m @@ -102,6 +102,8 @@ static grpc_credentials *CertificatesAtPath(NSString *path, NSError **errorPtr) [super initWithChannel:grpc_secure_channel_create(credentials, host.UTF8String, args)]); } +// TODO(jcanizales): GRPCSecureChannel and GRPCUnsecuredChannel are just convenience initializers +// for GRPCChannel. Move them into GRPCChannel, which will make the following unnecessary. - (instancetype)initWithChannel:(grpc_channel *)unmanagedChannel { [NSException raise:NSInternalInconsistencyException format:@"use another initializer"]; return [self initWithHost:nil]; // silence warnings diff --git a/src/objective-c/GRPCClient/private/GRPCUnsecuredChannel.m b/src/objective-c/GRPCClient/private/GRPCUnsecuredChannel.m index 5decfba7e36..070a529629a 100644 --- a/src/objective-c/GRPCClient/private/GRPCUnsecuredChannel.m +++ b/src/objective-c/GRPCClient/private/GRPCUnsecuredChannel.m @@ -41,6 +41,8 @@ return (self = [super initWithChannel:grpc_insecure_channel_create(host.UTF8String, NULL)]); } +// TODO(jcanizales): GRPCSecureChannel and GRPCUnsecuredChannel are just convenience initializers +// for GRPCChannel. Move them into GRPCChannel, which will make the following unnecessary. - (instancetype)initWithChannel:(grpc_channel *)unmanagedChannel { [NSException raise:NSInternalInconsistencyException format:@"use the other initializer"]; return [self initWithHost:nil]; // silence warnings