[phobos] [D-Programming-Language/phobos] 9fab54: Fix issue 10773.

GitHub noreply at github.com
Fri Aug 9 11:01:57 PDT 2013


  Branch: refs/heads/master
  Home:   https://github.com/D-Programming-Language/phobos
  Commit: 9fab541993c846fd5ddffedfa4a2014634b88cfc
      https://github.com/D-Programming-Language/phobos/commit/9fab541993c846fd5ddffedfa4a2014634b88cfc
  Author: H. S. Teoh <hsteoh at quickfur.ath.cx>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M std/algorithm.d

  Log Message:
  -----------
  Fix issue 10773.


  Commit: cd9696a8e81aa58eaa60adf1c95632519c32676a
      https://github.com/D-Programming-Language/phobos/commit/cd9696a8e81aa58eaa60adf1c95632519c32676a
  Author: H. S. Teoh <hsteoh at quickfur.ath.cx>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M std/algorithm.d

  Log Message:
  -----------
  Add missing signature constraints for splitter.

As well as require bidirectional separator with length before assuming
retro(separator) and separator.length will work.

Without these additional constraints, splitter will cause unhelpful
compile errors about being unable instantiate find, when the separator
is a non-forward input range, or when it doesn't have length, or when
it's non-bidirectional. In the latter case, splitter should still work,
just that it won't export a bidirectional interface.


  Commit: 78b9a6fd008e31c21e7aed77bb97c43c12387862
      https://github.com/D-Programming-Language/phobos/commit/78b9a6fd008e31c21e7aed77bb97c43c12387862
  Author: H. S. Teoh <hsteoh at quickfur.ath.cx>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M std/algorithm.d

  Log Message:
  -----------
  Don't need to loop over subranges to use equal().

I wrote it that way 'cos if the subrange is a distinct type from the
original range, equal won't compare them properly (compiler will
complain about == not being defined).

But in this case, it does work, so I rewrote it to use equal() directly.


  Commit: 395b72204ebc62f2788ed307f002824cbebaccdd
      https://github.com/D-Programming-Language/phobos/commit/395b72204ebc62f2788ed307f002824cbebaccdd
  Author: monarch dodra <monarchdodra at gmail.com>
  Date:   2013-08-09 (Fri, 09 Aug 2013)

  Changed paths:
    M std/algorithm.d

  Log Message:
  -----------
  Merge pull request #1456 from quickfur/issue10773

Fix issue 10773.


Compare: https://github.com/D-Programming-Language/phobos/compare/86514b0015ee...395b72204ebc


More information about the phobos mailing list