https://issues.dlang.org/show_bug.cgi?id=6106

Andrei Alexandrescu <andrei@erdani.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |bootcamp
                 CC|                            |andrei@erdani.com

--- Comment #1 from Andrei Alexandrescu <andrei@erdani.com> ---
This seems to have merit. Perhaps an optional last ref size_t parameter would fit the bill.

--

https://issues.dlang.org/show_bug.cgi?id=6106

RazvanN <razvan.nitu1305@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |razvan.nitu1305@gmail.com

--- Comment #2 from RazvanN <razvan.nitu1305@gmail.com> ---
(In reply to Andrei Alexandrescu from comment #1)
> This seems to have merit. Perhaps an optional last ref size_t parameter would fit the bill.

You can't have optional ref parameters. At the moment the language only supports default parameters (which we consider optional), but having a ref size_t parameter would imply something along the lines of foo(ref size_t a = value), where value would be the address of the parameter. We can still implement this by overloading the function

--

https://issues.dlang.org/show_bug.cgi?id=6106

--- Comment #3 from Andrei Alexandrescu <andrei@erdani.com> ---
"Optional" here means "an overload with 4 parameters"

--

https://issues.dlang.org/show_bug.cgi?id=6106

dcuan17@yahoo.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dcuan17@yahoo.com
           Assignee|nobody@puremagic.com        |dcuan17@yahoo.com

--

https://issues.dlang.org/show_bug.cgi?id=6106

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
@gizmomogwai updated dlang/phobos pull request #8694 "Try to implement https://issues.dlang.org/show_bug.cgi?id=6106" fixing this issue:

- fix Issue 6106 - Keep track of changes during replace function

  The functions without the newly introduced parameter forward
  to the full implementation to reduce code duplication.

https://github.com/dlang/phobos/pull/8694

--

https://issues.dlang.org/show_bug.cgi?id=6106

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/phobos pull request #8694 "fix Issue 6106 - Keep track of changes during replace function" was merged into master:

- 69be5ca021246cc01b20be98d10f9886dd0625fb by Christian Koestlin:
  fix Issue 6106 - Keep track of changes during replace function

  The functions without the newly introduced parameter forward
  to the full implementation to reduce code duplication.

https://github.com/dlang/phobos/pull/8694

--