search

EsotericSoftware / kryonet

TCP/UDP client/server library for Java, based on Kryo
BSD 3-Clause "New" or "Revised" License
1.82k stars 419 forks source link

Added IsListener interface (#39) #91

Closed highwaycoder closed 9 years ago

highwaycoder commented 9 years ago

I wanted Listener to be an interface, so I made an interface called IsListener. I'm not the only one, it seems, as there's an open issue that can probably be closed if this gets merged down.

WIP, needs testing.

NathanSweet commented 9 years ago

I won't merge anything that is not tested.

IsListener is not a good name.

I don't think this PR (or one like it) is necessary. If you really want an interface, just write a Listener that delegates to your interface.

Usually that a class listens to something is private information, jamming it on the class exposes information unnecessarily.