[m-rev.] For review: added any_list, any_map, any_array, any_assoc_list to extras
Julien Fischer
juliensf at cs.mu.OZ.AU
Sun Sep 11 15:50:30 AEST 2005
On Fri, 9 Sep 2005, Ralph Becket wrote:
> Estimated hours taken: 24
>
> Add list, map, assoc_list, and array ADTs for collections of solver type
> values.
>
> extras/solver_types:
> extras/solver_types/library:
> extras/solver_types/library/Mmakefile:
> extras/solver_types/library/any.m:
> extras/solver_types/library/any_map.m:
> extras/solver_types/library/any_array.m:
> extras/solver_types/library/any_tree234.m:
> extras/solver_types/library/any_assoc_list.m:
> extras/solver_types/library/any_list.m:
> Added.
>
> extras/solver_types/library/any_util.m:
> Defines unsafe_cast_to_ground/1 which is used very carefully
> in any_{map,tree234,assoc_list}.m to cast polymorphic key
> values from inst any to ground. This is necessary because
> the code needs to compare key values, but they have a polymorphic type
> and the compiler cannot statically infer that any = ground for
> polymorphically typed variables.
>
You also need to update extras/README.
> Index: extras/solver_types/library/Mmakefile
> ===================================================================
> RCS file: extras/solver_types/library/Mmakefile
> diff -N extras/solver_types/library/Mmakefile
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ extras/solver_types/library/Mmakefile 9 Sep 2005 05:57:47 -0000
> @@ -0,0 +1,21 @@
> +#-----------------------------------------------------------------------------#
> +# Copyright (C) 2000, 2002-2003 The University of Melbourne.
> +# This file may only be copied under the terms of the GNU Library General
> +# Public License - see the file COPYING.LIB in the Mercury distribution.
> +#-----------------------------------------------------------------------------#
> +
The years are wrong in the copyright messages and on the other files as
well (unless you started working on any_arrays in 1993).
That looks fine otherwise.
Cheers,
Julien.
--------------------------------------------------------------------------
mercury-reviews mailing list
post: mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------
More information about the reviews
mailing list