#687 also added regression test

Author: jantje

io.sloeber.core/src/io/sloeber/core/tools/PdePreprocessor.java

@@ -126,7 +126,7 @@ public static void processProject(IProject iProject) throws CoreException {
 								addString = addString.replaceAll("\r", NEWLINE);
 								addString = addString.replaceAll("//[^\n]+\n", " ");
 								addString = addString.replaceAll("\n", " ");
-								addString = addString.replaceAll("\\{.+\\}", "");
+								addString = addString.replaceAll("\\{.*\\}", "");
 								if (addString.contains("=") || addString.contains("::")) {
 									// ignore when there are assignments in the
 									// declaration

io.sloeber.core/src/jUnit/Regression.java

@@ -115,6 +115,36 @@ public void issue555() {
 		}
 	}
 
+	/**
+	 * support void loop{};
+	 */
+	@SuppressWarnings("static-method")
+	@Test
+	public void issue687() {
+		Map<String, String> unoOptions = new HashMap<>();
+		BoardDescriptor unoBoardid = BoardsManager.getBoardID("package_index.json", "arduino", "Arduino AVR Boards",
+				"uno", unoOptions);
+
+		IProject theTestProject = null;
+		String projectName = "issue687";
+		IPath templateFolder = Shared.getTemplateFolder(projectName);
+		CodeDescriptor codeDescriptor = CodeDescriptor.createCustomTemplate(templateFolder);
+		try {
+			theTestProject = unoBoardid.createProject(projectName, null,
+					ConfigurationDescriptor.getDefaultDescriptors(), codeDescriptor, new CompileOptions(null),
+					new NullProgressMonitor());
+			Shared.waitForAllJobsToFinish(); // for the indexer
+			theTestProject.build(IncrementalProjectBuilder.FULL_BUILD, new NullProgressMonitor());
+			if (Shared.hasBuildErrors(theTestProject)) {
+				fail("Failed to compile the project:" + projectName + " issue687 is not fixed");
+			}
+		} catch (Exception e) {
+			fail("Failed to create the project:" + projectName + " issue687 is not tested");
+			return;
+		}
+
+	}
+
 	/**
 	 * This test will fail if the arduino compile option are not taken into
 	 * account To do sa a bunch of defines are added to the command line and the

io.sloeber.core/src/jUnit/templates/issue687/sketch.ino

@@ -0,0 +1,10 @@
+#include "Arduino.h"
+
+
+
+//The setup function is with spaces between the curly braces
+void setup(){ }
+
+// The loop function is without content between curly braces
+void loop(){}
+