[GitHub] commons-collections issue #25: COLLECTIONS-575: Add synchronized queue wrapp...

2018-01-03 Thread Xaerxess
Github user Xaerxess commented on the issue:

https://github.com/apache/commons-collections/pull/25
  
As per comment in 
[COLLECTIONS-575](https://issues.apache.org/jira/browse/COLLECTIONS-575?focusedCommentId=16309068&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16309068),
 I'm closing this PR (merged in c6dc370abbbf0b487a13bd7f564287a41755bf71).


---

-
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org



[GitHub] commons-collections issue #25: COLLECTIONS-575: Add synchronized queue wrapp...

2017-09-11 Thread kinow
Github user kinow commented on the issue:

https://github.com/apache/commons-collections/pull/25
  
Hi @Xaerxess, had a few minutes to look at the pull request tonight. The 
code looks neat and simple to understand. Well indented, with tests, and 
comments. The Javadocs look well written too.

Can't thorough review and merge yet. But just wanted to give an initial 
feedback. As I am not well familiar with the [collections] code, and not 
actively working on it, will wait until someone else has time to properly 
review and maybe merge it.

In case it takes too long, feel free to mention my user name and I will try 
to program a couple hours to read through the code and try to check if there is 
anything missing in the PR, or if we need to get feedback from other via the 
mailing list.

Thanks a lot for your contribution. A great first PR :-)

Cheers,
Bruno


---

-
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org