diff --git a/core/src/main/java/org/lflang/target/TargetConfig.java b/core/src/main/java/org/lflang/target/TargetConfig.java index 6170947db0..96ce5efc9e 100644 --- a/core/src/main/java/org/lflang/target/TargetConfig.java +++ b/core/src/main/java/org/lflang/target/TargetConfig.java @@ -173,10 +173,9 @@ protected void load(JsonObject jsonObject, MessageReporter messageReporter) { * @param stage2 The second stage an the error reporter through which to report the warning. */ public void reportUnsupportedTargetProperty(String name, MessageReporter.Stage2 stage2) { - stage2.warning( + stage2.error( String.format( - "The target property '%s' is not supported by the %s target and is thus ignored.", - name, this.target)); + "The target property '%s' is not supported by the %s target.", name, this.target)); stage2.info("Recognized properties are: " + this.listOfRegisteredProperties()); } diff --git a/core/src/test/java/org/lflang/tests/compiler/LinguaFrancaValidationTest.java b/core/src/test/java/org/lflang/tests/compiler/LinguaFrancaValidationTest.java index 11fff6c2c9..41e68fa038 100644 --- a/core/src/test/java/org/lflang/tests/compiler/LinguaFrancaValidationTest.java +++ b/core/src/test/java/org/lflang/tests/compiler/LinguaFrancaValidationTest.java @@ -1905,11 +1905,11 @@ public void testInvalidTargetParam() throws Exception { var model = parseWithoutError(testCase); List issues = validator.validate(model); Assertions.assertTrue(issues.size() == 2); - validator.assertWarning( + validator.assertError( model, LfPackage.eINSTANCE.getKeyValuePair(), null, - "The target property 'foobarbaz' is not supported by the C target and is thus ignored."); + "The target property 'foobarbaz' is not supported by the C target."); } @Test @@ -1922,12 +1922,11 @@ public void testTargetParamNotSupportedForTarget() throws Exception { var model = parseWithoutError(testCase); List issues = validator.validate(model); Assertions.assertTrue(issues.size() == 2); - validator.assertWarning( + validator.assertError( model, LfPackage.eINSTANCE.getKeyValuePair(), null, - "The target property 'cargo-features' is not supported by the Python target and is thus" - + " ignored."); + "The target property 'cargo-features' is not supported by the Python target."); } @Test diff --git a/test/C/src/concurrent/Threaded.lf b/test/C/src/concurrent/Threaded.lf index 88a9acd6b6..a9598ca610 100644 --- a/test/C/src/concurrent/Threaded.lf +++ b/test/C/src/concurrent/Threaded.lf @@ -6,8 +6,7 @@ // that without threads, this takes more than 800 msec to complete 200 msec of logical time. See // ThreadedMultiport for a parameterized version of this. target C { - timeout: 2 sec, - flags: "" // Disable compiler optimization so that TakeTime actually takes time. + timeout: 2 sec } reactor Source { diff --git a/test/C/src/concurrent/ThreadedMultiport.lf b/test/C/src/concurrent/ThreadedMultiport.lf index 6fce42f0e9..cc09d36868 100644 --- a/test/C/src/concurrent/ThreadedMultiport.lf +++ b/test/C/src/concurrent/ThreadedMultiport.lf @@ -1,7 +1,6 @@ // Check multiport capabilities on Outputs. target C { - timeout: 2 sec, - flags: "" // Disable compiler optimization so that TakeTime actually takes time. + timeout: 2 sec } reactor Source(width: int = 4) { diff --git a/test/C/src/concurrent/ThreadedThreaded.lf b/test/C/src/concurrent/ThreadedThreaded.lf index 0b011b3118..6ad5204e02 100644 --- a/test/C/src/concurrent/ThreadedThreaded.lf +++ b/test/C/src/concurrent/ThreadedThreaded.lf @@ -6,8 +6,7 @@ // of this. target C { timeout: 2 sec, - tracing: true, - flags: "" // Disable compiler optimization so that TakeTime actually takes time. + tracing: true } reactor Source { diff --git a/test/C/src/concurrent/Tracing.lf b/test/C/src/concurrent/Tracing.lf index 983c9e737a..e0b0deb41b 100644 --- a/test/C/src/concurrent/Tracing.lf +++ b/test/C/src/concurrent/Tracing.lf @@ -2,7 +2,6 @@ target C { timeout: 2 sec, tracing: true, - flags: "", // Disable compiler optimization so that TakeTime actually takes time. logging: DEBUG } diff --git a/test/C/src/federated/LoopDistributedCentralizedPhysicalAction.lf b/test/C/src/federated/LoopDistributedCentralizedPhysicalAction.lf index ac783f07cc..59931b7f2c 100644 --- a/test/C/src/federated/LoopDistributedCentralizedPhysicalAction.lf +++ b/test/C/src/federated/LoopDistributedCentralizedPhysicalAction.lf @@ -4,7 +4,6 @@ * @author Edward A. Lee */ target C { - flags: "-Wall", coordination: centralized, coordination-options: { advance-message-interval: 100 msec diff --git a/test/C/src/federated/LoopDistributedCentralizedPrecedence.lf b/test/C/src/federated/LoopDistributedCentralizedPrecedence.lf index 51a10faac2..6b495a29d9 100644 --- a/test/C/src/federated/LoopDistributedCentralizedPrecedence.lf +++ b/test/C/src/federated/LoopDistributedCentralizedPrecedence.lf @@ -6,7 +6,6 @@ * @author Soroush Bateni */ target C { - flags: "-Wall", coordination: centralized, coordination-options: { advance-message-interval: 100 msec diff --git a/test/C/src/federated/LoopDistributedCentralizedPrecedenceHierarchy.lf b/test/C/src/federated/LoopDistributedCentralizedPrecedenceHierarchy.lf index 82adfca699..9d33a7ec1f 100644 --- a/test/C/src/federated/LoopDistributedCentralizedPrecedenceHierarchy.lf +++ b/test/C/src/federated/LoopDistributedCentralizedPrecedenceHierarchy.lf @@ -6,7 +6,6 @@ * @author Soroush Bateni */ target C { - flags: "-Wall", coordination: centralized, coordination-options: { advance-message-interval: 100 msec