On Mon, 10 Mar 2008, Massimo Belgrano wrote:
> Done. It is my fisrt contribute please control
> Now harbour have 4658 test instead 4455

Thank you but you haven't updated ChangeLog file.
Please read doc/howtosvn.txt first then update ChangeLog file
and remember about adding ChangeLog description during commit.
Please also update formatting. We are using 3 SPACEs for indenting
and TABs are forbidden. Look at other lines in this files and try
to keep the same formatting. When many people update the same file
using similar formatting greatly improve code readability.

> ! 2318 MAIN_TRANS(529) Transform(stod("19920509"), "@E")       
>        Result: "09.05.1992"
>      Expected: "09/05/92"

This line needs default date format before execution:
   SET DATEFORMAT TO "MM/DD/YY"
Please update this line to expected result

> ! 2721 NEW_STRINGS(484 hb_ValToStr( 4.0 / 2 )                  
>        Result: "         2.0000"
>      Expected: "         2.00"

You restored _SET_FIXED and/or _SET_DECIMAL value after transform tests.
Please add it or it will effect other tests.

> ! 2723 NEW_STRINGS(486 hb_ValToStr( hb_SToD( "20010101" ) )    
>        Result: "01-01-2001"
>      Expected: "2001.01.01"

You haven't restored _SET_DATEFORMAT value after transform tests.
Please add it or it will effect other tests.

best regards,
Przemek
_______________________________________________
Harbour mailing list
[email protected]
http://lists.harbour-project.org/mailman/listinfo/harbour

Reply via email to