Просмотр исходного кода

Nitpick plugin/manager.go: use loop to create directories

Instead of duplicating the same if condition per plugin manager directory,
use one if condition and a for-loop.

Signed-off-by: Boaz Shuster <ripcurld.github@gmail.com>
Boaz Shuster 7 лет назад
Родитель
Сommit
d75f1d8487
1 измененных файлов с 4 добавлено и 8 удалено
  1. 4 8
      plugin/manager.go

+ 4 - 8
plugin/manager.go

@@ -107,14 +107,10 @@ func NewManager(config ManagerConfig) (*Manager, error) {
 	manager := &Manager{
 	manager := &Manager{
 		config: config,
 		config: config,
 	}
 	}
-	if err := os.MkdirAll(manager.config.Root, 0700); err != nil {
-		return nil, errors.Wrapf(err, "failed to mkdir %v", manager.config.Root)
-	}
-	if err := os.MkdirAll(manager.config.ExecRoot, 0700); err != nil {
-		return nil, errors.Wrapf(err, "failed to mkdir %v", manager.config.ExecRoot)
-	}
-	if err := os.MkdirAll(manager.tmpDir(), 0700); err != nil {
-		return nil, errors.Wrapf(err, "failed to mkdir %v", manager.tmpDir())
+	for _, dirName := range []string{manager.config.Root, manager.config.ExecRoot, manager.tmpDir()} {
+		if err := os.MkdirAll(dirName, 0700); err != nil {
+			return nil, errors.Wrapf(err, "failed to mkdir %v", dirName)
+		}
 	}
 	}
 
 
 	if err := setupRoot(manager.config.Root); err != nil {
 	if err := setupRoot(manager.config.Root); err != nil {