| Store | Cart

Re: [TCLCORE] There is no justification for modifying [proc] for named parameters

From: Mathieu Lafon <mla...@gmail.com>
Sat, 20 May 2017 16:06:31 +0200
Colin,

My intent was not to denigrate your analysis but to try to calm things
down. Sorry for having triggered the opposite...

We have a disagreement on how this should have been implemented. I
have chosen to modify proc argument specifiers after having proposed
other options (including a dedicated command to call a proc :
http://www.tcl.tk/cgi-bin/tct/tip/view/457?ver=1.7) and received
feedback in that way. The initial FA bounty description was requiring
that it must work with unmodified proc, so the spirit of the bounty
seems to favor keeping proc and not adding a separate command.

Having chosen to modify proc, this implies impact on code, performance
and documentation, which I have tried to keep minimal, but are
inherent to that decision. I have carefully ensured that the new code
is well separated so that it is easy to review and maintain and that
the proc not using extended specifiers are not impacted in any way.

Regarding the two proposals made by Alexandre :

- Adding a new FAproc command will require to duplicate a lot of code
already present in proc, leading to more maintenance. Users may find
confusing to have a separate command and new users may not find it.

- Adding a runtime handler seems to be limited to argument parsing,
where the current proposal is a base framework which can be easily
extended to later support other usecase on proc arguments (type
assertion, doc-string, ...). Unless I don't get the intent, it seems
necessary to specify the named arguments when calling the command.
Users may find counter-intuitive to have to specify the arguments in a
second time (I assume proc only use 'args' in that case).

I don't have any strong positions on any alternate proposal, I only
want to give my current proposal a chance to be considered.

-- Mathieu


On Sat, May 20, 2017 at 12:41 PM, Colin McCormack <mcc...@gmail.com> wrote:
> Excuse me, Mathieu, but I'm not simply "against it.">> I have raised what I consider to be cogent criticisms of a proposed course> of action, and as far as I can tell you have addressed none of them.  I> sought to understand why our opinions differ about what is best.>> To denigrate my analysis as 'flaming' doesn't discredit it, and to fail to> meaningfully engage with it surely only undermines your proposal.>> I believe that you have misread the bounty's stated requirements, if you> believe that modifying [proc] is essential to satisfying them.>> Whence did the idea arise that distorting proc was the best way to add the> functionality you appear to desire, or desire to implement?>> Colin>>> On Sat, 20 May 2017, 20:02 Mathieu Lafon, <mla...@gmail.com> wrote:>>>> I don't want to enter a flame-war with you. I understand that you're>> against it and I completely respect your point of view but I don't>> think I will change your mind.>>>> I admit that I have failed to reach a consensus but please respect the>> work done and the opinion of the people who have supported the work.>>>> -- Mathieu

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Tcl-Core mailing list
Tcl-...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tcl-core

Recent Messages in this Thread
Colin McCormack May 20, 2017 03:11 am
Mathieu Lafon May 20, 2017 08:38 am
Colin McCormack May 20, 2017 08:53 am
Colin McCormack May 20, 2017 08:58 am
Mathieu Lafon May 20, 2017 10:09 am
Colin McCormack May 20, 2017 10:15 am
Colin McCormack May 20, 2017 10:16 am
Colin McCormack May 20, 2017 10:23 am
Mathieu Lafon May 20, 2017 10:31 am
Colin McCormack May 20, 2017 10:41 am
Colin McCormack May 20, 2017 10:54 am
Colin McCormack May 20, 2017 10:54 am
Mathieu Lafon May 20, 2017 02:06 pm
Colin McCormack May 20, 2017 02:53 pm
Alexandre Ferrieux May 21, 2017 12:01 am
Colin McCormack May 21, 2017 01:16 am
Colin McCormack May 21, 2017 02:04 am
Mathieu Lafon May 21, 2017 11:27 am
Alexandre Ferrieux May 21, 2017 01:06 pm
Kevin Kenny May 20, 2017 02:01 pm
Colin McCormack May 20, 2017 02:39 pm
Colin McCormack May 20, 2017 11:58 pm
Alexandre Ferrieux May 20, 2017 09:12 am
Colin McCormack May 20, 2017 09:45 am
Peter da Silva May 22, 2017 11:42 am
Colin McCormack May 22, 2017 12:05 pm
Messages in this thread