Thread overview
[phobos] [D-Programming-Language/phobos] 6c289a: Fix unnecessary template bloat in enforce.
Jul 22, 2012
GitHub
Jul 23, 2012
Martin Nowak
Jul 23, 2012
Jonathan M Davis
July 21, 2012
  Branch: refs/heads/master
  Home:   https://github.com/D-Programming-Language/phobos
  Commit: 6c289a89699426c86ddce8a22050c5d7a4057485
      https://github.com/D-Programming-Language/phobos/commit/6c289a89699426c86ddce8a22050c5d7a4057485
  Author: jmdavis <jmdavisProg@gmx.com>
  Date:   2012-07-21 (Sat, 21 Jul 2012)

  Changed paths:
    M std/exception.d

  Log Message:
  -----------
  Fix unnecessary template bloat in enforce.

The version of enforce which takes the file and line number as template arguments has been scheduled for deprecation and has been replaced with one which takes them as function arguments. The only code that this will effect is code which explicitly passes the file or line number to enforce.


  Commit: 304d94f50446ac46da645604bb15a58a0d67c684
      https://github.com/D-Programming-Language/phobos/commit/304d94f50446ac46da645604bb15a58a0d67c684
  Author: Jonathan M Davis <jmdavisProg@gmx.com>
  Date:   2012-07-21 (Sat, 21 Jul 2012)

  Changed paths:
    M std/exception.d

  Log Message:
  -----------
  Merge pull request #709 from jmdavis/enforce

Fix unnecessary template bloat in enforce.


Compare: https://github.com/D-Programming-Language/phobos/compare/17ba4bb5940d...304d94f50446


July 23, 2012
On Sun, 22 Jul 2012 05:55:56 +0200, GitHub <noreply@github.com> wrote:

>   Branch: refs/heads/master
>   Home:   https://github.com/D-Programming-Language/phobos
>   Commit: 6c289a89699426c86ddce8a22050c5d7a4057485
>       https://github.com/D-Programming-Language/phobos/commit/6c289a89699426c86ddce8a22050c5d7a4057485
>   Author: jmdavis <jmdavisProg@gmx.com>
>   Date:   2012-07-21 (Sat, 21 Jul 2012)
>
>   Changed paths:
>     M std/exception.d
>
>   Log Message:
>   -----------
>   Fix unnecessary template bloat in enforce.
>
> The version of enforce which takes the file and line number as template
> arguments has been scheduled for deprecation and has been replaced with
> one which takes them as function arguments. The only code that this will
> effect is code which explicitly passes the file or line number to
> enforce.
>
It's probably worth to reevaluate the usefulness of this template special case.
http://dlang.org/template.html#TemplateValueParameter
_______________________________________________
phobos mailing list
phobos@puremagic.com
http://lists.puremagic.com/mailman/listinfo/phobos

July 23, 2012
On Monday, July 23, 2012 23:31:49 Martin Nowak wrote:
> It's probably worth to reevaluate the usefulness of this template special
> case.
> http://dlang.org/template.html#TemplateValueParameter

I think that there are a few cases where it's useful (e.g. where you're dealing with a template which _isn't_ a function), so having it's probably a good idea. However, using it when default function parameters would work is _not_ a good idea.

- Jonathan M Davis
_______________________________________________
phobos mailing list
phobos@puremagic.com
http://lists.puremagic.com/mailman/listinfo/phobos