It is better to use simple form without else.
Signed-off-by: Hu Keping <hukeping@huawei.com>
| ... | ... |
@@ -26,13 +26,14 @@ func (cli *DockerCli) CmdLogout(args ...string) error {
|
| 26 | 26 |
|
| 27 | 27 |
if _, ok := cli.configFile.AuthConfigs[serverAddress]; !ok {
|
| 28 | 28 |
fmt.Fprintf(cli.out, "Not logged in to %s\n", serverAddress) |
| 29 |
- } else {
|
|
| 30 |
- fmt.Fprintf(cli.out, "Remove login credentials for %s\n", serverAddress) |
|
| 31 |
- delete(cli.configFile.AuthConfigs, serverAddress) |
|
| 29 |
+ return nil |
|
| 30 |
+ } |
|
| 32 | 31 |
|
| 33 |
- if err := cli.configFile.Save(); err != nil {
|
|
| 34 |
- return fmt.Errorf("Failed to save docker config: %v", err)
|
|
| 35 |
- } |
|
| 32 |
+ fmt.Fprintf(cli.out, "Remove login credentials for %s\n", serverAddress) |
|
| 33 |
+ delete(cli.configFile.AuthConfigs, serverAddress) |
|
| 34 |
+ if err := cli.configFile.Save(); err != nil {
|
|
| 35 |
+ return fmt.Errorf("Failed to save docker config: %v", err)
|
|
| 36 | 36 |
} |
| 37 |
+ |
|
| 37 | 38 |
return nil |
| 38 | 39 |
} |