[mono-vb] FileSystem patch and tests

Jambunathan K kjambunathan@novell.com
Thu, 17 Jun 2004 19:26:38 +0530


Rob

I have committed your changes. 

I have done some reformatting to FileSystem.cs so that it 
looks more pleasant. Kindly submit your further patches 
on the newly committed FileSystem.cs.

While submitting your next patch make sure that you adhere 
strictly to the coding guidelines for if, else and switch 
blocks and that you break up lines that have more than 134
chars.

Submit patches promptly as and when it becomes self-contained, 
so that manageability becomes easier.

I haven't taken time off to critically review your patches,
instinctively I can say that they are good.

By submitting patches and that too with consistency (inspite 
of two weeks being too short a time to arrive at a conclusion)
you are gradually moving from a casual contributor to a serious 
mono-vb developer.

So some more patches & enhancements ;-) , you will be on your way to
commit files directly to cvs.


Regards,
Jambunathan K.


On Thu, 2004-06-17 at 10:49, Rob.Tillie@Student.tUL.EDU wrote:
> Hello,
> 
> This patch:
> - replaces all leading spaces by tabs
> - Reimplementation / check against MainSoft code / reformat of first 5
> methods.
> - adds unit tests for these methods.
> 
> There remains one issue with these methods (affects 2 methods), but I
> haven't figured out how this behaviour can be achieved exactly using normal
> classes.
> 
> Could someone plz commit? 
> 
> Greetz,
> -- Rob.
>