Richard S. Hall wrote:
Alex Karasulu wrote:
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?

Yes, you could just detect whether stdin is invalid first, but you would need a thread if stdin is valid. Is there a proper way to determine if stdin is valid?

Hmmm this behavoir is truly unexpected since jsvc does some strange things to daemonize the process. I don't really know how this would manifest itself in the Java space. Basically the stdin file descriptor is closed in which case the String line will return null all the time (just a suspicion from looking at the code and the observed behavoir). I think rigging in the ability to switch the profile and the configuration file when not in debug mode is the key.

Alex

Reply via email to