Path fixes

Torsten Anders torstenanders at gmx.de
Tue Feb 13 17:13:04 CET 2007


Dear all,

> However, I'd like you to hold off on these until I can review.

No problem, I really appreciate it.

>  Also, I don't know if it's practical, but these are not small 
> changes, and it is easier to
> review small, well documented changes. The ideal way to do this is by 
> using a
> branch where each commit makes a small well-documented change.  I 
> realize svn is
> not well suited to small short-lived branches, which is why ideally 
> I'd really
> prefer to use bzr (at least for this purpose).  Of course, I am not 
> asking you
> to redo these proposed changes just to suit my preferences, but it 
> might be nice
> to consider my suggestion in the future.
>
> Still, if you could also provide the documentation that might go in 
> the future
> commit message, that would be helpful ;-)

Sorry for not following any standard procedure, I didn't know how this 
is usually expected to happen.     I am not at all experienced in this 
kind of thing, so please bear with me. Do you suggest I do a svn branch 
the next time? How did this happen before?

The little temporary package provided (Path.tgz) was intended to serve 
as a simple branch. Of course, there is no edit history. Nevertheless, 
I added some temporary explanation for the most  important changes.

I would certainly have preferred small changes only. I suggest you 
first run the test cases I provided with the original Path definition 
to get an idea why I had to change so much ;-) I wonder whether anybody 
tried, e.g., basename or dirname before, and I got lots of error 
messages when I used MacOS.

I also added some documentation remarks in the code which will in the 
end be added to the Path documentation *.sgml files (and then removed 
in the code). For example, there are several lines like

% [add to documentation] here goes my additional explanation

I thought it is more practical to have that in the code directly before 
things get accepted.

> Sure, and it is very important that when you fix bugs or change 
> behaviour you add test cases.  See mozart/share/test/.

I added tests in Path.tgz. These have to be run and interpreted 
manually however (currently at least). What is the standard format for 
Mozart test cases? Is there perhaps some documentation about that.


> ideally, someone should contribute a unit testing framework that we 
> could reuse :-) ahem...

Juergen Stueber actually did that for Mozart in 2004. I think he 
announced that on the mailing list. I still have his sources on my 
machine. It may not be fit for every purpose, but it would be a start..

Thank you very much!

Best,
Torsten

PS: I am currently not subscribed on the hackers mailing list and hope 
this gets through anyway..

--
Torsten Anders
Sonic Arts Research Centre • Queen's University Belfast
Frankstr. 49 • D-50996 Köln
Tel: +49-221-3980750
http://strasheela.sourceforge.net
http://www.torsten-anders.de



More information about the mozart-hackers mailing list