elharo commented on code in PR #1203:
URL: https://github.com/apache/maven/pull/1203#discussion_r1276627756


##########
api/maven-api-model/src/main/mdo/maven.mdo:
##########
@@ -2503,27 +2501,23 @@
     /**
      * Reset the {@code executionMap} field to {@code null}
      */
-    public void flushExecutionMap()
-    {
+    public void flushExecutionMap() {
         this.executionMap = null;
     }
+
     /**
      * @return a Map of executions field with {@code PluginExecution#getId()} 
as key
      * @see PluginExecution#getId()
      */
-    public java.util.Map<String, PluginExecution> getExecutionsAsMap()
-    {
-        if ( executionMap == null )
-        {
+    public java.util.Map<String, PluginExecution> getExecutionsAsMap() {

Review Comment:
   I would expect the code generator to figure out what imports it needs and do 
the right thing. If it doesn't, I guess that's another reason not to use a code 
generator. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to