[Mono-dev] [PATCH] System.Diagnostics/Process fix

Bill Holmes billholmes54 at gmail.com
Tue Sep 9 16:56:10 EDT 2008


> If it's not abusing of your good will. Please add some comments to
> Process.cs, Sockets.cs and socket-io.h mentioning this
> needs to be sync'ed in 3 places.

I was thinking the same thing.  I will add that.

> About stripping the '\r', won't that cause problems on unix machines?

I will have to check.  The quick answer is that I don't think so.  I
can remove this part of the patch for now as it is not mission
critical to me.  I simply found it while I was debugging the first
problem.

To recap.

I will add the comments.
remove the \r checks
commit to the trunk and branch
look at the \r problem another day.

-bill


More information about the Mono-devel-list mailing list