[GRASS5] Status of 5.0.0 release

Markus Neteler neteler at geog.uni-hannover.de
Tue Apr 10 03:37:31 EDT 2001


Hi Glynn,

(cc to grass5)

thanks for your various fixes!

Concerning the qsort warnings:

On Mon, Apr 09, 2001 at 07:58:22PM +0100, Glynn Clements wrote:
[...]
> Do you want the qsort() warnings fixed? AFAICT, all of these arise
> from comparison functions taking pointers to the actual object type,
> but qsort() having a fixed prototype with a fixed 4th parameter.
> 
> Fixing them is basically a choice of either:
> 
> a) changing e.g.:
> 
> 	int compare(foo *a, foo *b)
> 	{
> 		return CMP(*a, *b);
> 	}
> to
> 	int compare(void *aa, void *bb)
> 	{
> 		foo *a = (foo*) aa;
> 		foo *b = (foo*) bb;
> 		return CMP(*a, *b);
> 	}
> 
> or:
> 
> b) explicitly casting the 4th argument to qsort()
> 
> If you want this fixed, which approach would you prefer?
> 
> OTOH, the "fixed" code is possibly uglier than the code which
> generates warnings, due to the explicit typecasts.
> 
> -- 
> Glynn Clements <glynn.clements at virgin.net>
> 

this I would like to put into general discussion. Maybe we can silently
ignore the qsort warnings (no problems in past)?

Markus

---------------------------------------- 
If you want to unsubscribe from GRASS Development Team mailing list write to:
minordomo at geog.uni-hannover.de with
subject 'unsubscribe grass5'



More information about the grass-dev mailing list