[GitHub] maven-surefire pull request #171: [SUREFIRE-1445] Explicitly define Surefire...

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] maven-surefire pull request #171: [SUREFIRE-1445] Explicitly define Surefire...

Tibor17-2
Github user Tibor17 commented on a diff in the pull request:

    https://github.com/apache/maven-surefire/pull/171#discussion_r153654392
 
    --- Diff: maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/SurefireProperties.java ---
    @@ -71,13 +71,22 @@ public SurefireProperties( KeyValueSource source )
             }
         }
     
    +    @Override
    +    public synchronized void putAll(Map<?, ?> t) {
    +        for (Map.Entry<?, ?> entry : t.entrySet()) {
    +            this.put(entry.getKey(), entry.getValue());
    +        }
    +    }
    +
         @Override
         public synchronized Object put( Object key, Object value )
         {
             items.add( key );
             return super.put( key, value );
         }
     
    +
    +
    --- End diff --
   
    Here only one empty line between two methods. Feel free to remove these two you have added. Thx.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]