Removed DebugPreprocessor from builder ctx

Author: cmaglie

legacy/builder/container_add_prototypes.go

@@ -67,8 +67,7 @@ func PreprocessSketchWithCtags(ctx *types.Context) error {
 	if err != nil {
 		return err
 	}
-	ctx.SketchSourceAfterArduinoPreprocessing = PrototypesAdder(
-		ctx.Sketch, ctx.SketchSourceMerged, ctagsStdout, ctx.LineOffset, ctx.DebugPreprocessor)
+	ctx.SketchSourceAfterArduinoPreprocessing = PrototypesAdder(ctx.Sketch, ctx.SketchSourceMerged, ctagsStdout, ctx.LineOffset)
 
 	if err := bldr.SketchSaveItemCpp(ctx.Sketch.MainFile, []byte(ctx.SketchSourceAfterArduinoPreprocessing), ctx.SketchBuildPath); err != nil {
 		return errors.WithStack(err)

legacy/builder/prototypes_adder.go

@@ -26,7 +26,9 @@ import (
 	"github.com/arduino/arduino-cli/legacy/builder/utils"
 )
 
-func PrototypesAdder(sketch *sketch.Sketch, source string, ctagsStdout []byte, lineOffset int, debugOutput bool) string {
+var DebugPreprocessor bool
+
+func PrototypesAdder(sketch *sketch.Sketch, source string, ctagsStdout []byte, lineOffset int) string {
 	parser := &ctags.CTagsParser{}
 	parser.Parse(ctagsStdout, sketch.MainFile)
 	parser.FixCLinkageTagsDeclarations()
@@ -48,7 +50,7 @@ func PrototypesAdder(sketch *sketch.Sketch, source string, ctagsStdout []byte, l
 	prototypeSection := composePrototypeSection(firstFunctionLine, prototypes)
 	preprocessedSource := source[:firstFunctionChar] + prototypeSection + source[firstFunctionChar:]
 
-	if debugOutput {
+	if DebugPreprocessor {
 		fmt.Println("#PREPROCESSED SOURCE")
 		prototypesRows := strings.Split(prototypeSection, "\n")
 		prototypesRows = prototypesRows[:len(prototypesRows)-1]

legacy/builder/test/try_build_of_problematic_sketch_test.go

@@ -214,8 +214,8 @@ func makeDefaultContext(t *testing.T) *types.Context {
 		OtherLibrariesDirs:   paths.NewPathList("libraries"),
 		FQBN:                 parseFQBN(t, "arduino:avr:leonardo"),
 		Verbose:              true,
-		DebugPreprocessor:    true,
 	}
+	builder.DebugPreprocessor = true
 	buildPath := SetupBuildPath(t, ctx)
 	defer buildPath.RemoveAll()
 

legacy/builder/types/context.go

@@ -124,8 +124,7 @@ type Context struct {
 	LineOffset int
 
 	// Verbosity settings
-	Verbose           bool
-	DebugPreprocessor bool
+	Verbose bool
 
 	// Compile optimization settings
 	OptimizeForDebug  bool