You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2021/11/05 07:40:49 UTC

[GitHub] [maven-indexer] laeubi opened a new pull request #142: Update IndexDataReader.java

laeubi opened a new pull request #142:
URL: https://github.com/apache/maven-indexer/pull/142


   Issue reported here: https://github.com/eclipse-m2e/m2e-core/issues/193#issuecomment-961522331


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

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



[GitHub] [maven-indexer] cstamas commented on pull request #142: Update IndexDataReader.java

Posted by GitBox <gi...@apache.org>.
cstamas commented on pull request #142:
URL: https://github.com/apache/maven-indexer/pull/142#issuecomment-1027732028


   Superseded by https://github.com/apache/maven-indexer/pull/166


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

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



[GitHub] [maven-indexer] kmcmurtrieca commented on a change in pull request #142: Update IndexDataReader.java

Posted by GitBox <gi...@apache.org>.
kmcmurtrieca commented on a change in pull request #142:
URL: https://github.com/apache/maven-indexer/pull/142#discussion_r743882445



##########
File path: indexer-core/src/main/java/org/apache/maven/index/updater/IndexDataReader.java
##########
@@ -65,7 +65,7 @@ public IndexDataReader( final InputStream is )
         else
         {
             BufferedInputStream bis = new BufferedInputStream( is, 1024 * 8 );
-            bis.reset();
+            is.reset();

Review comment:
       Incorrect order of code.  Unused temporary variable.




-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

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



[GitHub] [maven-indexer] kmcmurtrieca commented on a change in pull request #142: Update IndexDataReader.java

Posted by GitBox <gi...@apache.org>.
kmcmurtrieca commented on a change in pull request #142:
URL: https://github.com/apache/maven-indexer/pull/142#discussion_r743882445



##########
File path: indexer-core/src/main/java/org/apache/maven/index/updater/IndexDataReader.java
##########
@@ -65,7 +65,7 @@ public IndexDataReader( final InputStream is )
         else
         {
             BufferedInputStream bis = new BufferedInputStream( is, 1024 * 8 );
-            bis.reset();
+            is.reset();

Review comment:
       Incorrect order of code.  Unused temporary variable.




-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

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



[GitHub] [maven-indexer] kmcmurtrieca commented on a change in pull request #142: Update IndexDataReader.java

Posted by GitBox <gi...@apache.org>.
kmcmurtrieca commented on a change in pull request #142:
URL: https://github.com/apache/maven-indexer/pull/142#discussion_r743882445



##########
File path: indexer-core/src/main/java/org/apache/maven/index/updater/IndexDataReader.java
##########
@@ -65,7 +65,7 @@ public IndexDataReader( final InputStream is )
         else
         {
             BufferedInputStream bis = new BufferedInputStream( is, 1024 * 8 );
-            bis.reset();
+            is.reset();

Review comment:
       Incorrect order of code.  Unused temporary variable.




-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

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



[GitHub] [maven-indexer] cstamas closed pull request #142: Update IndexDataReader.java

Posted by GitBox <gi...@apache.org>.
cstamas closed pull request #142:
URL: https://github.com/apache/maven-indexer/pull/142


   


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

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