You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pr@jena.apache.org by GitBox <gi...@apache.org> on 2021/04/01 19:14:54 UTC

[GitHub] [jena] kinow commented on a change in pull request #977: Various small cleanup things.

kinow commented on a change in pull request #977:
URL: https://github.com/apache/jena/pull/977#discussion_r605887553



##########
File path: jena-core/src/main/java/org/apache/jena/util/iterator/WrappedIterator.java
##########
@@ -31,78 +32,90 @@ a WrappedIterator is an ExtendedIterator wrapping around a plain (or
 public class WrappedIterator<T> extends NiceIterator<T>
     {
     /**
-         set to <code>true</code> if this wrapping doesn't permit the use of 
+         set to <code>true</code> if this wrapping doesn't permit the use of
          .remove(). Otherwise the .remove() is delegated to the base iterator.
     */
     protected boolean removeDenied;
-    
+
     /**
         Answer an ExtendedIterator returning the elements of <code>it</code>.
         If <code>it</code> is itself an ExtendedIterator, return that; otherwise
         wrap <code>it</code>.
     */
     public static <T> ExtendedIterator<T> create( Iterator<T> it )
         { return it instanceof ExtendedIterator<?> ? (ExtendedIterator<T>) it : new WrappedIterator<>( it, false ); }
-    
+
     /**
         Answer an ExtendedIterator wrapped round <code>it</code> which does not
         permit <code>.remove()</code> even if <code>it</code> does.
     */
     public static <T> WrappedIterator<T> createNoRemove( Iterator<T> it )
         { return new WrappedIterator<>( it, true ); }
-   
-    
+
+    /**
+     * Answer an ExtendedIterator wrapped round a {@link Stream} The extended

Review comment:
       missing period before `The extended`?

##########
File path: .github/workflows/maven.yml
##########
@@ -1,10 +1,11 @@
 ## Licensed under the terms of http://www.apache.org/licenses/LICENSE-2.0
 
 name: Apache Jena CI
+# Manual
+on: workflow_dispatch
 
-on:
-  push:
-    branches: [ main ]

Review comment:
       Not useful to have it running automatically?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org