Typos and documentation

bap-builder/DockerMode.go

@@ -22,8 +22,7 @@ func BuildDockerImage(cmdLine *BuildImageCmdLineArgs, contextPath string) error
 	if err != nil {
 		return err
 	}
-	buildSingleDockerImage(*cmdLine.Name, dockerfilePath)
-	return nil
+	return buildSingleDockerImage(*cmdLine.Name, dockerfilePath)
 }
 
 // buildAllDockerImages
@@ -42,7 +41,7 @@ func buildAllDockerImages(contextManager ContextManager) error {
 			logger.Warn("Bug: multiple Dockerfile present for same image name %s", imageName)
 			continue
 		}
-		buildSingleDockerImage(imageName, dockerfilePath[0])
+		return buildSingleDockerImage(imageName, dockerfilePath[0])
 	}
 	return nil
 }

modules/bringauto_docker/DockerTools.go

@@ -8,6 +8,9 @@ import (
 	"strconv"
 )
 
+// IsDefaultPortAvailable
+// Returns true if default port for docker is available, else returns false.
+// When false is returned, the error contains message from the docker command.
 func IsDefaultPortAvailable() (bool, error) {
 	var outBuff, errBuff bytes.Buffer
 

modules/bringauto_process/SignalHandler.go

@@ -68,9 +68,9 @@ func removeLastHandler() {
 
 func executeAllHandlers() {
 	for i := len(handlers)-1; i >= 0; i-- {
-   		err := handlers[i]()
-     	if err != nil {
-      		bringauto_log.GetLogger().Error("Handler returned error - %s", err)
-      	}
+		err := handlers[i]()
+		if err != nil {
+			bringauto_log.GetLogger().Error("Handler returned error - %s", err)
+		}
 	}
 }