refactor method visibility: make it public instead of protected

andimarek · andimarek · commit ce8caa197ad1 · 2018-03-17T21:14:04.000+11:00
diff --git a/src/main/java/graphql/util/RecursionState.java b/src/main/java/graphql/util/RecursionState.java
@@ -37,20 +37,20 @@ public TraverserContext<T> peek() {
 
     public abstract void pushAll(TraverserContext<T> o, Function<? super T, ? extends List<T>> getChildren);
 
-    protected void addAll(Collection<? extends T> col) {
+    public void addAll(Collection<? extends T> col) {
         assertNotNull(col).stream().map((x) -> newContext(x, null)).collect(Collectors.toCollection(() -> delegate));
     }
 
-    protected boolean isEmpty() {
+    public boolean isEmpty() {
         return delegate.isEmpty();
     }
 
-    protected void clear() {
+    public void clear() {
         delegate.clear();
         visitedMap.clear();
     }
 
-    protected TraverserContext<T> newContext(final T o, final TraverserContext<T> parent) {
+    public TraverserContext<T> newContext(final T o, final TraverserContext<T> parent) {
         return new TraverserContext<T>() {
             @Override
             public T thisNode() {
@@ -91,7 +91,4 @@ protected Deque<TraverserContext<?>> getDelegate() {
         return delegate;
     }
 
-    protected Map<T, Object> getVisitedMap() {
-        return visitedMap;
-    }
 }

Original file line number	Diff line number	Diff line change
`@@ -37,20 +37,20 @@ public TraverserContext<T> peek() {`
`37`	`37`
`38`	`38`	`public abstract void pushAll(TraverserContext<T> o, Function<? super T, ? extends List<T>> getChildren);`
`39`	`39`
`40`		`- protected void addAll(Collection<? extends T> col) {`
	`40`	`+ public void addAll(Collection<? extends T> col) {`
`41`	`41`	`assertNotNull(col).stream().map((x) -> newContext(x, null)).collect(Collectors.toCollection(() -> delegate));`
`42`	`42`	`}`
`43`	`43`
`44`		`- protected boolean isEmpty() {`
	`44`	`+ public boolean isEmpty() {`
`45`	`45`	`return delegate.isEmpty();`
`46`	`46`	`}`
`47`	`47`
`48`		`- protected void clear() {`
	`48`	`+ public void clear() {`
`49`	`49`	`delegate.clear();`
`50`	`50`	`visitedMap.clear();`
`51`	`51`	`}`
`52`	`52`
`53`		`- protected TraverserContext<T> newContext(final T o, final TraverserContext<T> parent) {`
	`53`	`+ public TraverserContext<T> newContext(final T o, final TraverserContext<T> parent) {`
`54`	`54`	`return new TraverserContext<T>() {`
`55`	`55`	`@Override`
`56`	`56`	`public T thisNode() {`
`@@ -91,7 +91,4 @@ protected Deque<TraverserContext<?>> getDelegate() {`
`91`	`91`	`return delegate;`
`92`	`92`	`}`
`93`	`93`
`94`		`- protected Map<T, Object> getVisitedMap() {`
`95`		`- return visitedMap;`
`96`		`- }`
`97`	`94`	`}`