[MonoDevelop] Win32 patch

John Luke john.luke at gmail.com
Thu Aug 31 23:06:47 EDT 2006


Hello,

On Thu, 2006-08-31 at 17:37 -0500, Levi Bard wrote:
> Comments?
> 
Is this for mono on windows or .net or both?

Also, there seems to be a bunch of whitespace changes which makes it
hard to review.

I didn't review the whole thing but here are some:
+ char splitc = SystemPackage.IsWin32 ? ';' : ':' ;

I think just use System.IO.Path.DirectorySeperatorChar for this

and

For all the path stuff like:
file = file.Substring (7);
if (file.StartsWith("/") && SystemPackage.IsWin32)
    file =file.Substring(1);

if (SystemPackage.IsWin32)
    datadir = datadir.Replace("\\","/");


it would probably be better if:
1) we just use '/' unconditionally on windows also (assuming that works
like someone told me it does)
2) we add helper methods (maybe in FileUtilityService?) to do that stuff
3) consistently use file:// Uris or regular paths and stop converting
to/from them all the time



More information about the Monodevelop-list mailing list