Use GRPC_ABSTRACT

src/core/ext/filters/client_channel/lb_policy.h

@@ -202,11 +202,7 @@ class LoadBalancingPolicy : public InternallyRefCounted<LoadBalancingPolicy> {
     /// by the client channel.
     virtual void UpdateState(grpc_connectivity_state state,
                              grpc_error* state_error,
-                             UniquePtr<SubchannelPicker>) {
+                             UniquePtr<SubchannelPicker>) GRPC_ABSTRACT;
-      // The rest of this is copied from the GRPC_ABSTRACT macro.
-      gpr_log(GPR_ERROR, "Function marked GRPC_ABSTRACT was not implemented");
-      GPR_ASSERT(false);
-    }
 
     /// Requests that the resolver re-resolve.
     virtual void RequestReresolution() GRPC_ABSTRACT;
@@ -260,9 +256,8 @@ class LoadBalancingPolicy : public InternallyRefCounted<LoadBalancingPolicy> {
   /// Note that the LB policy gets the set of addresses from the
   /// GRPC_ARG_SERVER_ADDRESS_LIST channel arg.
   virtual void UpdateLocked(const grpc_channel_args& args,
-                            RefCountedPtr<Config> lb_config) {  // NOLINT
+                            RefCountedPtr<Config>)  // NOLINT
-    GRPC_ABSTRACT;
+      GRPC_ABSTRACT;
-  }
 
   /// Tries to enter a READY connectivity state.
   /// This is a no-op by default, since most LB policies never go into

src/core/ext/filters/client_channel/lb_policy_factory.h

@@ -31,9 +31,7 @@ class LoadBalancingPolicyFactory {
  public:
   /// Returns a new LB policy instance.
   virtual OrphanablePtr<LoadBalancingPolicy> CreateLoadBalancingPolicy(
-      LoadBalancingPolicy::Args) const {
+      LoadBalancingPolicy::Args) const GRPC_ABSTRACT;
-    GRPC_ABSTRACT;
-  }
 
   /// Returns the LB policy name that this factory provides.
   /// Caller does NOT take ownership of result.