the Tcl implementation of lassign_array was removed from trunk and new tests were added for the same librivetlib implementation of the command. The manual now reflects the latter extended feature of returning the unassigned list values, as Tcl lassign does.

I have little time to dedicate on this thus I'm slowly proceeding one step after another towards a release candidate.

 -- Massimo


On 02/08/2014 12:57 AM, Massimo Manghi wrote:
Ronnie, you saved me from doing something potentially very stupid.
Thanks for pointing this out.

I always forget to consider rivetList.c and rivetCrypt.c because they're
not documented (unlike rivetWWW.c).

It seems to me the C code provides the extended feature of returning the
unassigned elements. I will check it more carefully and in case preserve
it dropping the Tcl procedure.

  DIO implementation was kept because it might exist code calling this
function as a member procedure of DIO derived class using the '$this
<method>' form.

  -- Massimo


The implementation in dio.tcl was kept

On 02/07/2014 03:11 PM, Ronnie Brunner wrote:
Hi Massimo

  4 command lassign_array was replicated from class ::DIO::Database
(being totally general and independent from member variables) and
promoted to core command

Plausible list except for - and that's my outside view ;-) - this
number 4: In Rivet I see a C implementation of lassign_array
(rivetList.c) and a Tcl implementation (so far) in dio.tcl. They don't
do the same thing. If you promote the dio version to core command,
you'll change the mechanics of the existing C implementation. Don't
you? Again: me not using Rivet doesn't really make me understand what
I'm saying ;-)

Best regards
Ronnie


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to