Merge pull request #6457 from laeubi/fix_maven_off_by_one

Fix of-by-one error in AbstractBndMavenPlugin
bndtools · Feb 5, 2025 · 1588a4c · 1588a4c
2 parents daf5ff0 + 5382286
commit 1588a4c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/...plugins/bnd-maven-plugin/src/main/java/aQute/bnd/maven/plugin/AbstractBndMavenPlugin.java b/...plugins/bnd-maven-plugin/src/main/java/aQute/bnd/maven/plugin/AbstractBndMavenPlugin.java
@@ -544,7 +544,8 @@ protected void reportErrorsAndWarnings(Builder builder) throws MojoFailureExcept
 			}
 			File f = location.file == null ? propertiesFile : new File(location.file);
 			markedFiles.add(f);
-			buildContext.addMessage(f, location.line, location.length, location.message, BuildContext.SEVERITY_WARNING,
+			buildContext.addMessage(f, location.line + 1, location.length, location.message,
+				BuildContext.SEVERITY_WARNING,
 				null);
 		}
 		List<String> errors = builder.getErrors();
@@ -556,7 +557,8 @@ protected void reportErrorsAndWarnings(Builder builder) throws MojoFailureExcept
 			}
 			File f = location.file == null ? propertiesFile : new File(location.file);
 			markedFiles.add(f);
-			buildContext.addMessage(f, location.line, location.length, location.message, BuildContext.SEVERITY_ERROR,
+			buildContext.addMessage(f, location.line + 1, location.length, location.message,
+				BuildContext.SEVERITY_ERROR,
 				null);
 		}
 		buildContext.setValue(MARKED_FILES, markedFiles);