| Store | Cart

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

From: Colin McCormack <mcc...@gmail.com>
Sat, 20 May 2017 08:53:38 +0000
Hi Mathieu,

I don't believe you.  Anything which adds functionality to an optimally
implemented proc must add overhead to it.  You might argue that proc as
implemented is sub-optimal, in which case thank you for finding a way to
optimise it.  You might argue that you cannot measure any difference
between the optimal and the extended versions, in which case you need to
measure more closely.  You might argue that the difference is
insignificant, or that it's worth the cost, which begs the question of why
using proc to provide the extension is significant or not worth the cost.

If you're honestly arguing that you have two (otherwise identical) pieces
of code, one of which implements a discriminator and one of which doesn't
and the one which performs more computation does so with no additional
computation time, then either I have a fundamentally flawed view of what
computation is, or you do.

I simply cannot buy it.

Colin.

On Sat, 20 May 2017, 18:09 Mathieu Lafon, <mla...@gmail.com> wrote:

> Hello Colin,>> On Sat, May 20, 2017 at 5:11 AM, Colin McCormack <mcc...@gmail.com> wrote:> > This re-implementation will by its nature drag on proc, will by its> nature> > slow proc definitions which don't use the extended facilities specified> by> > AFProc. This overhead would arise and be imposed for no good reason, but> > simply because the bounty was poorly phrased.>> You seems to assume that the implementation add an overhead on proc> execution, especially for proc not using the new argument specifiers,> but this is *not* the case.>> Please re-read the current TIP, look at the implementation or do your> own testing, there is no performance issue.>> I have carefully ensured that proc, which are not using new argument> specifiers, are not impacted in any way. On the opposite, users which> replace their own Tcl-pure code handling named parameter by using the> new specifiers will have a performance gain.>> I am perfectly aware that some people are against this proposal. I> just hope that they do not have false or outdated assumptions on the> subject, as many parts have changed since the initial proposal.>> That's why I'm currently requesting a vote, so that I can have a clear> position of TCT members on whether this can go into core or whether> this should be handled completely differently.>> Deciding whether this should be handled in proc or handled differently> (new command, calling an extension inside proc, ...) is not something> on which we can reach a consensus (at least I have clearly failed to> achieve that). A decision has to be done.>> Regards.>> -- 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