You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@johnzon.apache.org by st...@apache.org on 2017/08/29 10:50:51 UTC

johnzon git commit: JOHNZON-104 JsonPatchBuilder must be reusable

Repository: johnzon
Updated Branches:
  refs/heads/master 0e207b0a5 -> c8b6c1fe0


JOHNZON-104 JsonPatchBuilder must be reusable

reported as JSON_PROCESSING_SPEC-84 which got moved to
https://github.com/javaee/json-processing-spec/issues/84


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

Branch: refs/heads/master
Commit: c8b6c1fe05f0d2240874dc374d1edbe4b15d9106
Parents: 0e207b0
Author: Mark Struberg <st...@apache.org>
Authored: Tue Aug 29 12:49:44 2017 +0200
Committer: Mark Struberg <st...@apache.org>
Committed: Tue Aug 29 12:49:44 2017 +0200

----------------------------------------------------------------------
 .../main/java/org/apache/johnzon/core/JsonPatchBuilderImpl.java  | 4 ----
 1 file changed, 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/johnzon/blob/c8b6c1fe/johnzon-core/src/main/java/org/apache/johnzon/core/JsonPatchBuilderImpl.java
----------------------------------------------------------------------
diff --git a/johnzon-core/src/main/java/org/apache/johnzon/core/JsonPatchBuilderImpl.java b/johnzon-core/src/main/java/org/apache/johnzon/core/JsonPatchBuilderImpl.java
index bc5815d..ada2db7 100644
--- a/johnzon-core/src/main/java/org/apache/johnzon/core/JsonPatchBuilderImpl.java
+++ b/johnzon-core/src/main/java/org/apache/johnzon/core/JsonPatchBuilderImpl.java
@@ -154,11 +154,7 @@ class JsonPatchBuilderImpl implements JsonPatchBuilder {
 
     @Override
     public JsonPatch build() {
-
-        // add operations to another list
-        // so we can clear and reuse the builder
         JsonPatchImpl patch = new JsonPatchImpl(new ArrayList<>(operations));
-        operations.clear();
 
         return patch;