From 7e90745b3c609d12dfc63ca6cdddac9446b3d5e5 Mon Sep 17 00:00:00 2001 From: Jorge Canizales Date: Wed, 5 Aug 2015 21:19:22 -0700 Subject: Document plan to merge the GRPCChannel subclasses --- src/objective-c/GRPCClient/private/GRPCSecureChannel.m | 2 ++ src/objective-c/GRPCClient/private/GRPCUnsecuredChannel.m | 2 ++ 2 files changed, 4 insertions(+) diff --git a/src/objective-c/GRPCClient/private/GRPCSecureChannel.m b/src/objective-c/GRPCClient/private/GRPCSecureChannel.m index d3c4d41a13..9b4b6768f8 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 5decfba7e3..070a529629 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 -- cgit v1.2.3