[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=474613&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-474613 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 26/Aug/20 02:48 Start Date: 26/Aug/20 02:48 Worklog Time Spent: 10m Work Description: kinow commented on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-680442438 Merged. Thanks @XenoAmess ! Bruno 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 474613) Time Spent: 4h 10m (was: 4h) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Affects Versions: 4.4 >Reporter: Jin Xu >Assignee: Bruno P. Kinoshita >Priority: Minor > Fix For: 4.5 > > Time Spent: 4h 10m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=474612&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-474612 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 26/Aug/20 02:47 Start Date: 26/Aug/20 02:47 Worklog Time Spent: 10m Work Description: kinow closed pull request #165: URL: https://github.com/apache/commons-collections/pull/165 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 474612) Time Spent: 4h (was: 3h 50m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Affects Versions: 4.4 >Reporter: Jin Xu >Assignee: Bruno P. Kinoshita >Priority: Minor > Fix For: 4.5 > > Time Spent: 4h > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=461445&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-461445 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 21/Jul/20 07:58 Start Date: 21/Jul/20 07:58 Worklog Time Spent: 10m Work Description: coveralls edited a comment on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-637488425 [![Coverage Status](https://coveralls.io/builds/32214840/badge)](https://coveralls.io/builds/32214840) Coverage remained the same at 90.122% when pulling **8f53f1c842ba33d27b6f00cbcc489542a54b082c on XenoAmess:refine_javadoc_and_fix_typos** into **fd462c155b0484a0ae918dcd1ebd44f351636ea8 on apache:master**. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 461445) Time Spent: 3h 50m (was: 3h 40m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 3h 50m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=461440&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-461440 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 21/Jul/20 07:38 Start Date: 21/Jul/20 07:38 Worklog Time Spent: 10m Work Description: XenoAmess edited a comment on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-661688227 @wlfbck > Do these fixes include that for example [CircularFifoQueue](https://commons.apache.org/proper/commons-collections/javadocs/api-4.4/org/apache/commons/collections4/queue/CircularFifoQueue.html) has dummydoc for `poll()`, `element()`, `peek()`, `remove()`? Because that stuff is not in the source, so i assume it is generated. But it hides the original javadoc from java.util.Queue, which is very annoying. Nope. But I think it shall be able to be solved by adding: ``` /** * {@inheritDoc} */ ``` The question you mentioned is planned to be fixed by using maven-javadoc-plugin. Currently the plan is: 1. wait for me fully recover from a flu.(not yet QAQ) 2. my pr about some basic refinement to maven-javadoc-plugin get reviewed and passed.(still waiting) 3. I will make a pr to fix some bugs in maven-javadoc-plugin's javadoc:fix operation, and see it get passed. 4. I use the javadoc:fix on this repo, and see what doc can be auto fixed. The reason I don't want to use the javadoc:fix now is it really have some bugs, I mean, quite some. But that will take lots of times to wait (maven guys are really...busy...) So if you really want some of the docs be fixed soon, you can apply javadoc:fix on this repo, and revert all things it messed up, only leave some good changes.(but that is also a human-time-coster). 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 461440) Time Spent: 3.5h (was: 3h 20m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 3.5h > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=461441&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-461441 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 21/Jul/20 07:38 Start Date: 21/Jul/20 07:38 Worklog Time Spent: 10m Work Description: XenoAmess edited a comment on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-661688227 @wlfbck > Do these fixes include that for example [CircularFifoQueue](https://commons.apache.org/proper/commons-collections/javadocs/api-4.4/org/apache/commons/collections4/queue/CircularFifoQueue.html) has dummydoc for `poll()`, `element()`, `peek()`, `remove()`? Because that stuff is not in the source, so i assume it is generated. But it hides the original javadoc from java.util.Queue, which is very annoying. Nope. But I think it shall be able to be solved by adding: ``` /** * {@inheritDoc} */ ``` The question you mentioned is planned to be fixed by using maven-javadoc-plugin. Currently the plan is: 1. wait for me fully recover from a flu.(not yet QAQ) 2. my pr about some basic refinement to maven-javadoc-plugin get reviewed and passed.(still waiting) 3. I will make a pr to fix some bugs in maven-javadoc-plugin's javadoc:fix operation, and see it get passed. 4. I use the javadoc:fix on this repo, and see what doc can be auto fixed. The reason I don't want to use the javadoc:fix now is it really have some bugs, I mean, quite some. But that will take lots of times to wait (maven guys are really...busy...) So if you really want some of the docs be fixed soon, you can apply javadoc:fix on this repo, and revert all things it messed up, only leave some good changes.(but that is also a human-time-coster). Anyway, it should be in another pr. This pr is already too large. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 461441) Time Spent: 3h 40m (was: 3.5h) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 3h 40m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=461439&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-461439 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 21/Jul/20 07:37 Start Date: 21/Jul/20 07:37 Worklog Time Spent: 10m Work Description: XenoAmess edited a comment on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-661688227 @wlfbck > Do these fixes include that for example [CircularFifoQueue](https://commons.apache.org/proper/commons-collections/javadocs/api-4.4/org/apache/commons/collections4/queue/CircularFifoQueue.html) has dummydoc for `poll()`, `element()`, `peek()`, `remove()`? Because that stuff is not in the source, so i assume it is generated. But it hides the original javadoc from java.util.Queue, which is very annoying. Nope. But I think it shall be able to be solved by adding: ``` /** * {@inheritDoc} */ ``` The question you mentioned is planned to be fixed by using maven-javadoc-plugin. Currently the plan is: 1. wait for me fully recover from a flu.(not yet QAQ) 2. my pr about some basic refinement to maven-javadoc-plugin get reviewed and passed.(still waiting) 3. I will make a pr to fix some bugs in maven-javadoc-plugin's javadoc:fix operation, and see it get passed. 4. I use the javadoc:fix on this repo, and see what doc can be auto fixed. The reason I don't want to use the javadoc:fix now is it really have some bugs, I mean, quite some. But that will take lots of times to wait (maven guys are really...busy...) So if you really want some of the docs be fixed soon, you can apply javadoc:fix on this repo, and revert all things it messed up, only leave some good changes.(but that is also a time coster). 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 461439) Time Spent: 3h 20m (was: 3h 10m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 3h 20m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=461436&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-461436 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 21/Jul/20 07:33 Start Date: 21/Jul/20 07:33 Worklog Time Spent: 10m Work Description: XenoAmess commented on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-661688227 @wlfbck > Do these fixes include that for example [CircularFifoQueue](https://commons.apache.org/proper/commons-collections/javadocs/api-4.4/org/apache/commons/collections4/queue/CircularFifoQueue.html) has dummydoc for `poll()`, `element()`, `peek()`, `remove()`? Because that stuff is not in the source, so i assume it is generated. But it hides the original javadoc from java.util.Queue, which is very annoying. Nope. But I think it shall be able to be solved by adding: ``` /** * {@inheritDoc} */ ``` The question you mentioned is planned to be fixed by using maven-javadoc-plugin. Currently the plan is: 1. wait for me fully recover from a flu.(not yet QAQ) 2. my pr about some basic refinement to maven-javadoc-plugin get reviewed and passed.(still waiting) 3. I will make a pr to fix some bugs in maven-javadoc-plugin's javadoc:fix operation, and see it get passed. 4. I use the javadoc:fix on this repo, and see what doc can be auto fixed. The reason I don't want to use the javadoc:fix now is it really have some bugs, I mean, quite some. But that will take lots of times to wait (maven guys a really...busy...) So if you really want some of the docs be fixed soon, you can apply javadoc:fix on it, and revert all things it messed up, only leave some good changes.(but that is also a time coster). 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 461436) Time Spent: 3h (was: 2h 50m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 3h > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=461437&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-461437 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 21/Jul/20 07:33 Start Date: 21/Jul/20 07:33 Worklog Time Spent: 10m Work Description: XenoAmess edited a comment on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-661688227 @wlfbck > Do these fixes include that for example [CircularFifoQueue](https://commons.apache.org/proper/commons-collections/javadocs/api-4.4/org/apache/commons/collections4/queue/CircularFifoQueue.html) has dummydoc for `poll()`, `element()`, `peek()`, `remove()`? Because that stuff is not in the source, so i assume it is generated. But it hides the original javadoc from java.util.Queue, which is very annoying. Nope. But I think it shall be able to be solved by adding: ``` /** * {@inheritDoc} */ ``` The question you mentioned is planned to be fixed by using maven-javadoc-plugin. Currently the plan is: 1. wait for me fully recover from a flu.(not yet QAQ) 2. my pr about some basic refinement to maven-javadoc-plugin get reviewed and passed.(still waiting) 3. I will make a pr to fix some bugs in maven-javadoc-plugin's javadoc:fix operation, and see it get passed. 4. I use the javadoc:fix on this repo, and see what doc can be auto fixed. The reason I don't want to use the javadoc:fix now is it really have some bugs, I mean, quite some. But that will take lots of times to wait (maven guys are really...busy...) So if you really want some of the docs be fixed soon, you can apply javadoc:fix on it, and revert all things it messed up, only leave some good changes.(but that is also a time coster). 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 461437) Time Spent: 3h 10m (was: 3h) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 3h 10m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=458645&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-458645 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 14/Jul/20 14:15 Start Date: 14/Jul/20 14:15 Worklog Time Spent: 10m Work Description: wlfbck commented on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-658205744 Do these fixes include that for example [CircularFifoQueue](https://commons.apache.org/proper/commons-collections/javadocs/api-4.4/org/apache/commons/collections4/queue/CircularFifoQueue.html) has dummydoc for `poll()`, `element()`, `peek()`, `remove()`? Because that stuff is not in the source, so i assume it is generated. But it hides the original javadoc from java.util.Queue, which is very annoying. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 458645) Time Spent: 2h 50m (was: 2h 40m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 2h 50m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=445565&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-445565 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 14/Jun/20 11:59 Start Date: 14/Jun/20 11:59 Worklog Time Spent: 10m Work Description: coveralls edited a comment on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-637488425 [![Coverage Status](https://coveralls.io/builds/31440627/badge)](https://coveralls.io/builds/31440627) Coverage remained the same at 90.123% when pulling **09f9fdde8ba8b5e83634255862c7f72757e37c5e on XenoAmess:refine_javadoc_and_fix_typos** into **6b4d75f725d9636b3ec6d5c24d410bece5e2fcc7 on apache:master**. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 445565) Time Spent: 2h 40m (was: 2.5h) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 2h 40m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=445563&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-445563 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 14/Jun/20 11:42 Start Date: 14/Jun/20 11:42 Worklog Time Spent: 10m Work Description: XenoAmess commented on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-643755003 the travis-ci false pending for several days already. will try to retrigger it. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 445563) Time Spent: 2.5h (was: 2h 20m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: Jin Xu >Priority: Minor > Time Spent: 2.5h > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440129&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440129 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 13:20 Start Date: 02/Jun/20 13:20 Worklog Time Spent: 10m Work Description: coveralls edited a comment on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-637488425 [![Coverage Status](https://coveralls.io/builds/31185352/badge)](https://coveralls.io/builds/31185352) Coverage remained the same at 90.123% when pulling **7f920b6b74c7e83f62230b8f372766f520f41ad4 on XenoAmess:refine_javadoc_and_fix_typos** into **6b4d75f725d9636b3ec6d5c24d410bece5e2fcc7 on apache:master**. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440129) Time Spent: 2h 20m (was: 2h 10m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 2h 20m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440109&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440109 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 12:46 Start Date: 02/Jun/20 12:46 Worklog Time Spent: 10m Work Description: XenoAmess commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433847044 ## File path: src/main/java/org/apache/commons/collections4/trie/PatriciaTrie.java ## @@ -29,7 +29,7 @@ * all data at the edges of the {@link org.apache.commons.collections4.Trie} * (and having empty internal nodes), PATRICIA stores data in every node. * This allows for very efficient traversal, insert, delete, predecessor, - * successor, prefix, range, and {@link #select(Object)} + * successor, prefix, range, and {@link #select(String)} Review comment: @dota17 Though I have no idea why it can pass on 8 but not others Strange isn't it... ![image](https://user-images.githubusercontent.com/17455337/83521545-f9ab4680-a511-11ea-8ff6-9a6c1734c46b.png) 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440109) Time Spent: 2h 10m (was: 2h) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 2h 10m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440107&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440107 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 12:45 Start Date: 02/Jun/20 12:45 Worklog Time Spent: 10m Work Description: XenoAmess commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433846044 ## File path: src/main/java/org/apache/commons/collections4/trie/PatriciaTrie.java ## @@ -29,7 +29,7 @@ * all data at the edges of the {@link org.apache.commons.collections4.Trie} * (and having empty internal nodes), PATRICIA stores data in every node. * This allows for very efficient traversal, insert, delete, predecessor, - * successor, prefix, range, and {@link #select(Object)} + * successor, prefix, range, and {@link #select(String)} Review comment: @dota17 yes you are correct. It can pass on one of my machines with jdk8 so I thought it be correct. but actually according to travis-ci, it can only pass openjdk8 and oraclejdk8, but fails on other jdks. so I will revert it. Thank you! 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440107) Time Spent: 2h (was: 1h 50m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 2h > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440105&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440105 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 12:38 Start Date: 02/Jun/20 12:38 Worklog Time Spent: 10m Work Description: XenoAmess commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433841232 ## File path: src/main/java/org/apache/commons/collections4/list/CursorableLinkedList.java ## @@ -279,7 +279,7 @@ protected void registerCursor(final Cursor cursor) { } /** - * Deregisters a cursor from the list to be notified of changes. + * Deregister a cursor from the list to be notified of changes. Review comment: @dota17 yes you are right. will revert it. --- btw, should I change it to `Unregister a cursor from the list to be notified of changes.`? because the function name used unregisterCursor, so I guess should we make it somehow compatibility? ## File path: src/main/java/org/apache/commons/collections4/list/CursorableLinkedList.java ## @@ -279,7 +279,7 @@ protected void registerCursor(final Cursor cursor) { } /** - * Deregisters a cursor from the list to be notified of changes. + * Deregister a cursor from the list to be notified of changes. Review comment: @dota17 yes you are right. will revert it. `---` btw, should I change it to `Unregister a cursor from the list to be notified of changes.`? because the function name used unregisterCursor, so I guess should we make it somehow compatibility? 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440105) Time Spent: 1h 50m (was: 1h 40m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 1h 50m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440104&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440104 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 12:36 Start Date: 02/Jun/20 12:36 Worklog Time Spent: 10m Work Description: XenoAmess commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433841232 ## File path: src/main/java/org/apache/commons/collections4/list/CursorableLinkedList.java ## @@ -279,7 +279,7 @@ protected void registerCursor(final Cursor cursor) { } /** - * Deregisters a cursor from the list to be notified of changes. + * Deregister a cursor from the list to be notified of changes. Review comment: yes you are right. will revert it. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440104) Time Spent: 1h 40m (was: 1.5h) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 1h 40m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440100&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440100 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 12:24 Start Date: 02/Jun/20 12:24 Worklog Time Spent: 10m Work Description: dota17 commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433834707 ## File path: src/main/java/org/apache/commons/collections4/trie/PatriciaTrie.java ## @@ -29,7 +29,7 @@ * all data at the edges of the {@link org.apache.commons.collections4.Trie} * (and having empty internal nodes), PATRICIA stores data in every node. * This allows for very efficient traversal, insert, delete, predecessor, - * successor, prefix, range, and {@link #select(Object)} + * successor, prefix, range, and {@link #select(String)} Review comment: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-javadoc-plugin:3.1.1:javadoc (default-cli) on project commons-collections4: An error has occurred in Javadoc report generation: [ERROR] Exit code: 1 - /home/travis/build/apache/commons-collections/src/main/java/org/apache/commons/collections4/trie/PatriciaTrie.java:32: error: reference not found [ERROR] * successor, prefix, range, and {@link #select(String)} Maybe it's not need to update. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440100) Time Spent: 1.5h (was: 1h 20m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 1.5h > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440096&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440096 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 12:17 Start Date: 02/Jun/20 12:17 Worklog Time Spent: 10m Work Description: dota17 commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433830911 ## File path: src/main/java/org/apache/commons/collections4/list/CursorableLinkedList.java ## @@ -279,7 +279,7 @@ protected void registerCursor(final Cursor cursor) { } /** - * Deregisters a cursor from the list to be notified of changes. + * Deregister a cursor from the list to be notified of changes. Review comment: garydgregory on 17 Apr Member If a method starts with "make...", it usually reads better to start the Javadoc with "Makes..." IMO, It's confusing to have the Javadoc be different, otherwise you could say that all method that start with "set..." also "forces" a value. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440096) Time Spent: 1h 20m (was: 1h 10m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 1h 20m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440081&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440081 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 11:50 Start Date: 02/Jun/20 11:50 Worklog Time Spent: 10m Work Description: coveralls commented on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-637488425 [![Coverage Status](https://coveralls.io/builds/31183231/badge)](https://coveralls.io/builds/31183231) Coverage decreased (-0.03%) to 90.094% when pulling **d4bcae0548d7fb3f3bcdc2eb99f152c3839197d9 on XenoAmess:refine_javadoc_and_fix_typos** into **6b4d75f725d9636b3ec6d5c24d410bece5e2fcc7 on apache:master**. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440081) Time Spent: 1h 10m (was: 1h) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 1h 10m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440079&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440079 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 11:49 Start Date: 02/Jun/20 11:49 Worklog Time Spent: 10m Work Description: kinow commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433816020 ## File path: src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java ## @@ -341,10 +341,6 @@ public V selectValue(final K key) { return entry.getValue(); } -/** - * This is equivalent to the other {@link #selectR(TrieEntry, int, Object, int, Cursor, Reference)} - * method but without its overhead because we're selecting only one best matching Entry from the {@link Trie}. - */ Review comment: Ah, you are correct! There was another method, but it was removed when the base class was renamed and the code changed a little bit to create this abstract class - https://github.com/apache/commons-collections/commit/83226e162de60bd22551b1a37cc3e513811e1449#diff-821b6e7d45ec0fd696f3cdc8027f029cL303 All good here then. Thanks! 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440079) Time Spent: 1h (was: 50m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 1h > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440062&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440062 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 11:08 Start Date: 02/Jun/20 11:08 Worklog Time Spent: 10m Work Description: XenoAmess commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433795336 ## File path: src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java ## @@ -341,10 +341,6 @@ public V selectValue(final K key) { return entry.getValue(); } -/** - * This is equivalent to the other {@link #selectR(TrieEntry, int, Object, int, Cursor, Reference)} - * method but without its overhead because we're selecting only one best matching Entry from the {@link Trie}. - */ Review comment: > I think this one needs to stay? @kinow I delete it because I can never found that function it point to. If you can show me where that function is : `selectR(TrieEntry, int, Object, int, Cursor, Reference)`, maybe I can fix the javadoc in a better way. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440062) Time Spent: 50m (was: 40m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 50m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440061&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440061 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 11:08 Start Date: 02/Jun/20 11:08 Worklog Time Spent: 10m Work Description: XenoAmess commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433795336 ## File path: src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java ## @@ -341,10 +341,6 @@ public V selectValue(final K key) { return entry.getValue(); } -/** - * This is equivalent to the other {@link #selectR(TrieEntry, int, Object, int, Cursor, Reference)} - * method but without its overhead because we're selecting only one best matching Entry from the {@link Trie}. - */ Review comment: > I think this one needs to stay? @kinow I delete it because I can never found that function it point to. If you can show me where that function is : `selectR(TrieEntry, int, Object, int, Cursor, Reference)`, maybe I can fix the javadoc in a better way. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440061) Time Spent: 40m (was: 0.5h) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 40m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440059&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440059 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 11:07 Start Date: 02/Jun/20 11:07 Worklog Time Spent: 10m Work Description: XenoAmess commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433795336 ## File path: src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java ## @@ -341,10 +341,6 @@ public V selectValue(final K key) { return entry.getValue(); } -/** - * This is equivalent to the other {@link #selectR(TrieEntry, int, Object, int, Cursor, Reference)} - * method but without its overhead because we're selecting only one best matching Entry from the {@link Trie}. - */ Review comment: > I think this one needs to stay? I delete it because I can never found that function it point to. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440059) Time Spent: 0.5h (was: 20m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 0.5h > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440056&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440056 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 10:55 Start Date: 02/Jun/20 10:55 Worklog Time Spent: 10m Work Description: kinow commented on a change in pull request #165: URL: https://github.com/apache/commons-collections/pull/165#discussion_r433788010 ## File path: src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java ## @@ -341,10 +341,6 @@ public V selectValue(final K key) { return entry.getValue(); } -/** - * This is equivalent to the other {@link #selectR(TrieEntry, int, Object, int, Cursor, Reference)} - * method but without its overhead because we're selecting only one best matching Entry from the {@link Trie}. - */ Review comment: I think this one needs to stay? 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440056) Time Spent: 20m (was: 10m) > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 20m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (COLLECTIONS-764) Refine javadoc and fix typos
[ https://issues.apache.org/jira/browse/COLLECTIONS-764?focusedWorklogId=440054&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-440054 ] ASF GitHub Bot logged work on COLLECTIONS-764: -- Author: ASF GitHub Bot Created on: 02/Jun/20 10:52 Start Date: 02/Jun/20 10:52 Worklog Time Spent: 10m Work Description: XenoAmess commented on pull request #165: URL: https://github.com/apache/commons-collections/pull/165#issuecomment-637457648 Several people suggested me not to put everything in one single pr, but split them. So I listened to their advice, and let things done one by one. 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: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 440054) Remaining Estimate: 0h Time Spent: 10m > Refine javadoc and fix typos > > > Key: COLLECTIONS-764 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-764 > Project: Commons Collections > Issue Type: Improvement >Reporter: JIN XU >Priority: Minor > Time Spent: 10m > Remaining Estimate: 0h > > [https://github.com/apache/commons-collections/pull/165] -- This message was sent by Atlassian Jira (v8.3.4#803005)