You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2017/12/08 18:15:04 UTC

[maven-mapping] 23/42: [MSHARED-452] Removed calling to accessor

This is an automated email from the ASF dual-hosted git repository.

hboutemy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-mapping.git

commit d1f9a8965b1ef161321d3c2ca459cba200fa3674
Author: Karl Heinz Marbaise <kh...@apache.org>
AuthorDate: Sat Oct 31 14:09:18 2015 +0000

    [MSHARED-452] Removed calling to accessor
    
    git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1711636 13f79535-47bb-0310-9956-ffa450edef68
---
 src/main/java/org/apache/maven/shared/mapping/MappingUtils.java | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/mapping/MappingUtils.java b/src/main/java/org/apache/maven/shared/mapping/MappingUtils.java
index 7667838..6668527 100644
--- a/src/main/java/org/apache/maven/shared/mapping/MappingUtils.java
+++ b/src/main/java/org/apache/maven/shared/mapping/MappingUtils.java
@@ -65,10 +65,6 @@ public class MappingUtils
     {
         String value = expression;
 
-        // FIXME: This is BAD! Accessors SHOULD NOT change the behavior of the object.
-        // [dennisl; 2013-07-30] This was fixed in Maven 2.0.8
-        artifact.isSnapshot();
-
         RegexBasedInterpolator interpolator = new RegexBasedInterpolator( "\\@\\{(", ")?([^}]+)\\}@" );
         interpolator.addValueSource( new ObjectBasedValueSource( artifact ) );
         interpolator.addValueSource( new ObjectBasedValueSource( artifact.getArtifactHandler() ) );

-- 
To stop receiving notification emails like this one, please contact
"commits@maven.apache.org" <co...@maven.apache.org>.