Hi,

A few random comments...:

1) org.apache.mina.statemachine.mina.Events could be replaced with
org.apache.mina.common.IoEventType.

2) WDYT about renaming MinaStateContextLookup to
IoSessionStateContextLookup and move it up to
org.apache.mina.statemachine?

3) We could mark mina-core dependency 'optional' because statemachine
module can work without mina-core.

Cheers,
Trustin
-- 
what we call human nature is actually human habit
--
http://gleamynode.net/
--
PGP Key ID: 0x0255ECA6

Reply via email to