search

gisle / tcl.pm

Tcl extension module for Perl
http://search.cpan.org/dist/Tcl
9 stars 8 forks source link

RT 132735: tie'ing Perl arrays to Tcl lists #38

Open chrstphrchvz opened 3 years ago

chrstphrchvz commented 3 years ago

Initial implementation as submitted by @marekro in https://rt.cpan.org/Ticket/Display.html?id=132735 and discussed further in #36.

chrstphrchvz commented 3 years ago

I agree that the important thing at this point probably is to get a new release out, as it's been almost 3 years since 1.27.

For the tests, I would consider using Test.pm or similar for maintainability, and maybe put them in a new file.

Thinking more about what delete on an element should do, it sounds like Perl already discourages its use with arrays, which to me sounds like it may not be a good idea to define it as a synonym for something convenient but very different from conventional Perl array behavior. Unless others are convinced this isn't an issue, I would probably prefer to leave delete unimplemented (leaving it an error to use it), and maybe put the proposed convenience functionality in a separate method.

If I were to propose using Tcl list commands instead, I'm not clear what effect the flags should have, other than the Tcl::GLOBAL_ONLY() flag bit meaning that any usage of $interp->Eval() should be passed the Tcl::EVAL_GLOBAL() flag.