CVS: fixed tests; minor manual editing

Bernhard Reiter bernhard at intevation.de
Thu Aug 11 16:51:34 CEST 2005


Hi Jakson,

yesterday I was in the train for a few hours and had a recent cvs
checkout with me, but not the proposed new announcement nor the webpage.

Here is the changelog of what I have checked into the CVS just now:

+       * Fixed the unit tests.
+         + tests/test_4_1.sh: fixed test: expecting new comment in the output.
+         + tests/check_memory_handling.c: fixed test for "check" v>=0.9.0.
+
+       * Minor editing of the English manual.
+         + Minor clean of doc/Makefile:
+           - Ignoring errors of rm.
+           - Declared phony targets .PHONY. Also rewrote "all" target.
+         + Called the operating system GNU/Linux as Linux is only a kernel.
+         + Made the message recommending Free Software friendlier and
+           more correct about the security implications.
+         + added the gnuplot method to create output files.
+         + updated the build and install instructions. "su -" in general
+           is prefered over "su".

Did you run the automatic tests during development?
Just say "make check" in src/ to run them.

Automatic tests are nice for development cycle to do 
test a little -- code a little
which is a method also used by extreme programming.


When compiling I also found a few interesting warning by the compiler:
        Comparision ist always true due to limited range of data type.
        in 
		data.c:1057
		recode.c:378
		recode.c:466

        This makes sense, as 'ÿ' == (char)255.
        So you could leave out the right part of the check.
        This would be different if char would be unicode character,
        which has more than 8 bits. I have no experience with unicode
        in modern C development, but I expect that the comparisons
        will be completely different anyway.

Best,
	Bernhard

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://www.intevation.de/pipermail/statist-list/attachments/20050811/f411ef3f/attachment.bin


More information about the Statist-list mailing list

This site is hosted by Intevation GmbH (Datenschutzerklärung und Impressum | Privacy Policy and Imprint)