hyperhq / runv

Hypervisor-based Runtime for OCI
Apache License 2.0
826 stars 129 forks source link

change func name StartServer into NewServer #601

Closed allencloud closed 7 years ago

allencloud commented 7 years ago

When I was learning runV, I found that in a place maybe func name NewServer is a little bit more proper than StartServer.

Since the function only initialize a grpc server with registered grpc service, util s.Serve(l) the grpc server starts to work.

Just with very junior background with runV, if I missed something here, please feel free to correct me. Thanks a lot.

Signed-off-by: Allen Sun shlallen1990@gmail.com

laijs commented 7 years ago

LGTM