Recent GH activities - 15:00 09-May-2012

[ne...@googlegroups.com]

jfarcand commented on issue 315 on netty/netty
==============================================
URL: https://github.com/netty/netty/issues/315#issuecomment-5599103

[jfarcand][1] commented on [issue 315][2] on [netty/netty][3] May 09, 2012

> OK 3.4.4 fixed the issue. Thanks!!!!!

[1]: https://github.com/jfarcand
[2]: https://github.com/netty/netty/issues/315#issuecomment-5599103 (java.lang.StackOverflowError with 3.4.3)
[3]: https://github.com/netty/netty

jfarcand closed issue 315 on netty/netty
========================================
URL: https://github.com/netty/netty/issues/315

[jfarcand][1] closed [issue 315][2] on [netty/netty][3] May 09, 2012

> java.lang.StackOverflowError with 3.4.3

[1]: https://github.com/jfarcand
[2]: https://github.com/netty/netty/issues/315 (java.lang.StackOverflowError with 3.4.3)
[3]: https://github.com/netty/netty

trustin pushed to next-api at netty/netty
=========================================
URL: https://github.com/netty/netty/compare/607d784e5e...129a2af86a

[trustin][1] pushed to next-api at [netty/netty][2] May 09, 2012

* [trustin][1] committed [129a2af][3]

> Initial working version of the echo server example

- Optimized AbstractChannelBuffer.discardReadBytes()
- Split ChannelHandlerInvoker into ChannelInboundInvoker and
ChannelOutboundInvoker
- Channel implements ChannelOutboundInvoker
- ChannelOutboundInvoker.nextOut() is now out()
- ChannelOutboundHandlerContext.out() is now prevOut()
- Added the outbound operations without future
parameter to ChannelOutboundInvoker for user convenience
- All async operations which requires a ChannelFuture as a parameter
now returns ChannelFuture for user convenience
- Added ChannelFutureFactory.newVoidFuture() to allow a user specify
a dummy future that is of no use
- I'm unsure if it is actually a good idea to introduce it. It might
go away later.
- Made the contract of AbstractChannel.doXXX() much simpler and moved
all common code up to AbstractChannel.DefaultUnsafe
- Added Channel.isOpen()
- Fixed a bug where MultithreadEventLoop always shut down its child
event loops on construction
- Maybe more changes I don't remember :-)

[1]: https://github.com/trustin
[2]: https://github.com/netty/netty
[3]: https://github.com/netty/netty/commit/129a2af86adda8631eee2a5fe3c2b442a223a724

normanmaurer commented on netty/netty
=====================================
URL: https://github.com/netty/netty/commit/129a2af86adda8631eee2a5fe3c2b442a223a724#example/src/main/java/io/netty/example/echo/EchoServer.java-P40

[normanmaurer][1] commented on [netty/netty][2] May 09, 2012

Comment on [example/src/main/java/io/netty/example/echo/EchoServer.java][3] [L48][3] in [129a2af86a][3]:

> I must say I don't like this to much and all the reflection magic going in in the class ;) Why not inject a Factory which can create SelectorEventLoop instances and use this ?

[1]: https://github.com/normanmaurer
[2]: https://github.com/netty/netty
[3]: https://github.com/netty/netty/commit/129a2af86adda8631eee2a5fe3c2b442a223a724#example/src/main/java/io/netty/example/echo/EchoServer.java-P40

carrot-garden commented on issue 311 on netty/netty
===================================================
URL: https://github.com/netty/netty/issues/311#issuecomment-5600224

[carrot-garden][1] commented on [issue 311][2] on [netty/netty][3] May 09, 2012

> cool. thanks for considering this.

[1]: https://github.com/carrot-garden
[2]: https://github.com/netty/netty/issues/311#issuecomment-5600224 (daily snapshot?)
[3]: https://github.com/netty/netty

carrot-garden commented on issue 216 on netty/netty
===================================================
URL: https://github.com/netty/netty/issues/216#issuecomment-5600274

[carrot-garden][1] commented on [issue 216][2] on [netty/netty][3] May 09, 2012

> UDP support for NIO is primary motivation for me to go for netty 4; please let me know if this is feasible in netty 3.5

[1]: https://github.com/carrot-garden
[2]: https://github.com/netty/netty/issues/216#issuecomment-5600274 (Add support for UDP Multicast)
[3]: https://github.com/netty/netty

rpelissier commented on issue 314 on netty/netty
================================================
URL: https://github.com/netty/netty/issues/314#issuecomment-5601293

[rpelissier][1] commented on [issue 314][2] on [netty/netty][3] May 09, 2012

> Okay I will do it as soon as possible. I need to remove references to our own API... and I need to get through learning how to use Git/GitHub!

[1]: https://github.com/rpelissier
[2]: https://github.com/netty/netty/issues/314#issuecomment-5601293 (Would like to contribute with Dynamic FrameDecoder package)
[3]: https://github.com/netty/netty

trustin pushed to next-api at netty/netty
=========================================
URL: https://github.com/netty/netty/compare/129a2af86a...b4764f6164

[trustin][1] pushed to next-api at [netty/netty][2] May 09, 2012

* [trustin][1] committed [b4764f6][3]

> Fix infinity loop and timing issues

- Made sure unnecessary interestOps are not OR'd
- Fixed a bug where DefaultChannelFuture.rethrowIfFailed() returns
silently if the future is not done yet - there's no ways to tell
the differences between failure and incompleteness.

[1]: https://github.com/trustin
[2]: https://github.com/netty/netty
[3]: https://github.com/netty/netty/commit/b4764f61646ad56383eb884a6ca6e562b458aa65