[Aldor-l] remove: (T->Boolean, %) -> %

[Christian Aistleitner]

Hello Ralf,

>          remove: (T -> Boolean, %) -> %;
> 	remove!: (T -> Boolean, %) -> %;
> 	removeAll: (T -> Boolean, %) -> %;
> 	removeAll!: (T -> Boolean, %)
I suppose that's meant to read
  	removeAll!: (T -> Boolean, %) -> %;
>
> DynamicDataStructureType(T:Type):Category == ...
>
> and implement the corresponding functions in
> Queue, List, Set, SortedList, SortedSet.
>
[snip]
> Would there be anybody against if I add these functionalities and commit  
> them?

assuming the responsible persons decide to add these functions, I indeed  
have objections, if it is done in the presented way. Implementing these  
functions in Queue, List, Set, SortedList, and SortedSet is not enough.
I think the most important implementation has to go in the

default {
...
}

part of DynamicDataStructureType. Otherwise, every domain implementing  
DynamicDataStructureType requires updates. These updates are NOT  
restricted to the Aldor library, but affect all libraries that depend on  
the Aldor library as well.

Accompanying the optimized implementations for Queue, List, Set,  
SortedList, and SortedSet along with a general default implementation in  
DynamicDataStructureType, voids my objections.

But that is probably what you intended anyway, although you did not write  
it in explicitly.

--
Kind regards,
Christian