JM == James McCauley <James>

JM> Date: Tue, 31 Aug 2010 05:15:41 -0700

JM> This is a destiny-specific bug and was introduced in commit 
58547d74330eb0511cdddbbeb4f3017278a50d53.
JM>
JM> I am not set up for commits here, but I've attached a patch. I think
JM> there are actually a number of improvements that could be made to the
JM> socket-based logging, but that's a project for another day. At least
JM> this stops it from crashing.

Not crashing seems good, but truncating the log message seems clearly
wrong. Would splitting it into multiple log messages make more sense?

                                      -Josh (j...@bbn.com)

_______________________________________________
nox-dev mailing list
nox-dev@noxrepo.org
http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org

Reply via email to