Richard S. Hall wrote:
After looking closer at the patch I notice a couple of issues:

  1. Does it make sense to set the thread as a daemon? I don't think
     frameworks are required to have non-daemon threads...we would have
     to think about it.
Personally I don't understand why there is another thread? What's the point to it? Can't you just detect that stdin is not valid and just handle things in the same thread?

  2. Bundles are not supposed to call stop on themselves, so we would
     have to change that...we don't really need to stop the bundle, we
     can just have it stop processing input.
+1 to this ... that's the core problem.

Alex

Reply via email to