Use constant to replace integer literal

Signed-off-by: Hu Keping <hukeping@huawei.com>
This commit is contained in:
HuKeping 2016-04-22 11:08:42 -04:00
parent 165815fdbf
commit 10800f62ae
2 changed files with 10 additions and 10 deletions

View file

@ -43,7 +43,7 @@ func listCheckpoints(context *cli.Context) {
id = context.Args().First() id = context.Args().First()
) )
if id == "" { if id == "" {
fatal("container id cannot be empty", 1) fatal("container id cannot be empty", ExitStatusMissingArg)
} }
resp, err := c.ListCheckpoint(netcontext.Background(), &types.ListCheckpointRequest{ resp, err := c.ListCheckpoint(netcontext.Background(), &types.ListCheckpointRequest{
Id: id, Id: id,
@ -88,10 +88,10 @@ var createCheckpointCommand = cli.Command{
name = context.Args().Get(1) name = context.Args().Get(1)
) )
if containerID == "" { if containerID == "" {
fatal("container id at cannot be empty", 1) fatal("container id at cannot be empty", ExitStatusMissingArg)
} }
if name == "" { if name == "" {
fatal("checkpoint name cannot be empty", 1) fatal("checkpoint name cannot be empty", ExitStatusMissingArg)
} }
c := getClient(context) c := getClient(context)
if _, err := c.CreateCheckpoint(netcontext.Background(), &types.CreateCheckpointRequest{ if _, err := c.CreateCheckpoint(netcontext.Background(), &types.CreateCheckpointRequest{
@ -118,10 +118,10 @@ var deleteCheckpointCommand = cli.Command{
name = context.Args().Get(1) name = context.Args().Get(1)
) )
if containerID == "" { if containerID == "" {
fatal("container id at cannot be empty", 1) fatal("container id at cannot be empty", ExitStatusMissingArg)
} }
if name == "" { if name == "" {
fatal("checkpoint name cannot be empty", 1) fatal("checkpoint name cannot be empty", ExitStatusMissingArg)
} }
c := getClient(context) c := getClient(context)
if _, err := c.DeleteCheckpoint(netcontext.Background(), &types.DeleteCheckpointRequest{ if _, err := c.DeleteCheckpoint(netcontext.Background(), &types.DeleteCheckpointRequest{

View file

@ -146,10 +146,10 @@ var startCommand = cli.Command{
path = context.Args().Get(1) path = context.Args().Get(1)
) )
if path == "" { if path == "" {
fatal("bundle path cannot be empty", 1) fatal("bundle path cannot be empty", ExitStatusMissingArg)
} }
if id == "" { if id == "" {
fatal("container id cannot be empty", 1) fatal("container id cannot be empty", ExitStatusMissingArg)
} }
bpath, err := filepath.Abs(path) bpath, err := filepath.Abs(path)
if err != nil { if err != nil {
@ -336,7 +336,7 @@ var pauseCommand = cli.Command{
Action: func(context *cli.Context) { Action: func(context *cli.Context) {
id := context.Args().First() id := context.Args().First()
if id == "" { if id == "" {
fatal("container id cannot be empty", 1) fatal("container id cannot be empty", ExitStatusMissingArg)
} }
c := getClient(context) c := getClient(context)
_, err := c.UpdateContainer(netcontext.Background(), &types.UpdateContainerRequest{ _, err := c.UpdateContainer(netcontext.Background(), &types.UpdateContainerRequest{
@ -356,7 +356,7 @@ var resumeCommand = cli.Command{
Action: func(context *cli.Context) { Action: func(context *cli.Context) {
id := context.Args().First() id := context.Args().First()
if id == "" { if id == "" {
fatal("container id cannot be empty", 1) fatal("container id cannot be empty", ExitStatusMissingArg)
} }
c := getClient(context) c := getClient(context)
_, err := c.UpdateContainer(netcontext.Background(), &types.UpdateContainerRequest{ _, err := c.UpdateContainer(netcontext.Background(), &types.UpdateContainerRequest{
@ -388,7 +388,7 @@ var killCommand = cli.Command{
Action: func(context *cli.Context) { Action: func(context *cli.Context) {
id := context.Args().First() id := context.Args().First()
if id == "" { if id == "" {
fatal("container id cannot be empty", 1) fatal("container id cannot be empty", ExitStatusMissingArg)
} }
c := getClient(context) c := getClient(context)
if _, err := c.Signal(netcontext.Background(), &types.SignalRequest{ if _, err := c.Signal(netcontext.Background(), &types.SignalRequest{