Defaults to runtime.NumCPU() if specified jobs number is 0

Author: cmaglie

commands/compile/compile.go

@@ -23,7 +23,6 @@ import (
 	"fmt"
 	"io"
 	"path/filepath"
-	"runtime"
 	"sort"
 	"strings"
 
@@ -116,11 +115,7 @@ func Compile(ctx context.Context, req *rpc.CompileReq, outStream, errStream io.W
 
 	builderCtx.CoreBuildCachePath = paths.TempDir().Join("arduino-core-cache")
 
-	jobs := runtime.NumCPU()
-	if req.GetJobs() > 0 {
-		jobs = int(req.GetJobs())
-	}
-	builderCtx.Jobs = jobs
+	builderCtx.Jobs = int(req.GetJobs())
 
 	builderCtx.USBVidPid = req.GetVidPid()
 	builderCtx.WarningsLevel = req.GetWarnings()

legacy/builder/builder_utils/utils.go

@@ -33,6 +33,7 @@ import (
 	"os"
 	"os/exec"
 	"path/filepath"
+	"runtime"
 	"strconv"
 	"strings"
 	"sync"
@@ -192,7 +193,11 @@ func compileFilesWithRecipe(ctx *types.Context, sourcePath *paths.Path, sources
 
 	// Spawn jobs runners
 	var wg sync.WaitGroup
-	for i := 0; i < ctx.Jobs; i++ {
+	jobs := ctx.Jobs
+	if jobs == 0 {
+		jobs = runtime.NumCPU()
+	}
+	for i := 0; i < jobs; i++ {
 		wg.Add(1)
 		go func() {
 			for source := range queue {