[haiku-commits] Re: haiku: hrev50043 - src/servers/media_addon

  • From: Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 21 Jan 2016 13:56:19 +0100


It was more explicit before, actually. Now you will happily run into an
endless loop when an
unforeseen condition happens (like a B_NO_PERMISSION, for example).
Please revert -- this code is
not acceptable.
I think you are referring to some future behavior, and I actually can't
see why the loop should
exit on something different than the port close. The opposite situation
may happen, what if an
error code which isn't fatal makes the loop to exit?

At least that is easily detected and debugged. I prefer that the thread
stops, rather than going into an infinite loop (as the mail daemon
currently does on my machine for similar reasons...)


If we were in a generic situation, I agree completely. But contextualize
the specific situation: we are asking for an infinite timeout, from what I
understand, doing it means we will always block on the call until there's
something to read. Since this is the SystemTimeSource, and having it to
exit means losing media functionality in the whole system, I think it could
be more safe to make it stay up until the media_addon_server goes down.

Other related posts: