You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by bv...@apache.org on 2014/01/18 19:29:50 UTC

git commit: CAMEL-7142: Polished the comment.

Updated Branches:
  refs/heads/master 0d47ae108 -> 38e1367eb


CAMEL-7142: Polished the comment.

Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/38e1367e
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/38e1367e
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/38e1367e

Branch: refs/heads/master
Commit: 38e1367eba96fc3f00d6047b77a6872522afc67a
Parents: 0d47ae1
Author: Babak Vahdat <bv...@apache.org>
Authored: Sat Jan 18 19:29:41 2014 +0100
Committer: Babak Vahdat <bv...@apache.org>
Committed: Sat Jan 18 19:29:41 2014 +0100

----------------------------------------------------------------------
 .../java/org/apache/camel/dataformat/csv/CsvDataFormat.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/38e1367e/components/camel-csv/src/main/java/org/apache/camel/dataformat/csv/CsvDataFormat.java
----------------------------------------------------------------------
diff --git a/components/camel-csv/src/main/java/org/apache/camel/dataformat/csv/CsvDataFormat.java b/components/camel-csv/src/main/java/org/apache/camel/dataformat/csv/CsvDataFormat.java
index 64d9036..a409b77 100644
--- a/components/camel-csv/src/main/java/org/apache/camel/dataformat/csv/CsvDataFormat.java
+++ b/components/camel-csv/src/main/java/org/apache/camel/dataformat/csv/CsvDataFormat.java
@@ -70,9 +70,9 @@ public class CsvDataFormat implements DataFormat {
         for (Field field : CSVStrategy.class.getFields()) {
             try {
                 if (field.get(null) == csvStrategy) {
-                    // return a safe copy of the static constant so that we don't cause any side effect
-                    // by the other CsvDataFormat objects in case we would change any property on this
-                    // strategy itself (e.g. it's delimiter through the #unmarshal() method below)
+                    // return a safe copy of the declared constant so that we don't cause any side effect
+                    // by (potentially) other CsvDataFormat objects in use as we change properties of
+                    // the strategy (e.g. it's set delimiter through the #unmarshal() method below)
                     LOGGER.debug("Returning a clone of {} as it is the declared constant {} by the CSVStrategy class", csvStrategy, field.getName());
 
                     return (CSVStrategy) csvStrategy.clone();