https://issues.dlang.org/show_bug.cgi?id=19057
Iain Buclaw changed:
What|Removed |Added
Priority|P1 |P2
--
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #14 from Jonathan M Davis ---
The current behavior works well with default arguments, because then they never
get accidentally matched, whereas IFTI is allowed to match arguments against
them, then it becomes trivial to match them
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #13 from johanenge...@weka.io ---
(In reply to Jonathan M Davis from comment #11)
>
> I confess that I never would have expected parameters to be allowed after
> variadic arguments though, so the current behavior actually matches up
>
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #12 from Steven Schveighoffer ---
(In reply to Jonathan M Davis from comment #11)
> Well, having file or line end up being given values just because a string or
> string and integral value happened to be last in the argument list would
>
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #11 from Jonathan M Davis ---
(In reply to Steven Schveighoffer from comment #8)
> I think we need to find a way to change this behavior, and soon. As it
> stands now, it doesn't make sense, as it's only useful in the __FILE__ and
>
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #10 from johanenge...@weka.io ---
(In reply to Steven Schveighoffer from comment #9)
> (In reply to johanengelen from comment #7)
> > This did break code at Weka, luckily in non-silent way, but now I probably
> > will have to modify the
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #9 from Steven Schveighoffer ---
(In reply to johanengelen from comment #7)
> This did break code at Weka, luckily in non-silent way, but now I probably
> will have to modify the old compiler to error on these things, to not cause
>
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #8 from Steven Schveighoffer ---
I think we need to find a way to change this behavior, and soon. As it stands
now, it doesn't make sense, as it's only useful in the __FILE__ and __LINE__
context. It could easily be generalized to fit
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #7 from johanenge...@weka.io ---
Thanks for the support Steven.
I hope this helps put the message across to be _very careful_ in making
language additions / changes. This should have been a DIP. And not a PR that
gets merged after a
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #6 from Steven Schveighoffer ---
OK, I misunderstood. So essentially, any function that uses variadic parameters
AND a normal parameter with a default value was EXACTLY the same as a function
which takes the varargs parameter and a
https://issues.dlang.org/show_bug.cgi?id=19057
Jonathan M Davis changed:
What|Removed |Added
CC||issues.dl...@jmdavisprog.co
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #4 from johanenge...@weka.io ---
My concern is not about use cases. It's about breaking old code (silently), and
also about strange (arguable) language behavior in the current situation.
Note that before 2.079, the default argument was
https://issues.dlang.org/show_bug.cgi?id=19057
Seb changed:
What|Removed |Added
CC||greensunn...@gmail.com
--
https://issues.dlang.org/show_bug.cgi?id=19057
Steven Schveighoffer changed:
What|Removed |Added
See Also||https://issues.dlang.org/sh
https://issues.dlang.org/show_bug.cgi?id=19057
--- Comment #3 from Steven Schveighoffer ---
So the use case for the new style is pretty well discussed and documented (and
makes a lot of sense). The use cases for the old style are not discussed
really.
Looks like the only times default
https://issues.dlang.org/show_bug.cgi?id=19057
Mike Franklin changed:
What|Removed |Added
CC||slavo5...@yahoo.com
--- Comment #2 from
https://issues.dlang.org/show_bug.cgi?id=19057
Steven Schveighoffer changed:
What|Removed |Added
CC||schvei...@yahoo.com
--- Comment #1
https://issues.dlang.org/show_bug.cgi?id=19057
johanenge...@weka.io changed:
What|Removed |Added
Keywords||industry
--
18 matches
Mail list logo