Thread overview
[Issue 16993] Documentation for toSimpleString and toString does not explain how they differ
Jun 26, 2017
Vladimir Panteleev
Jul 11, 2017
RazvanN
Jul 11, 2017
Jonathan M Davis
December 22, 2016
https://issues.dlang.org/show_bug.cgi?id=16993

Andrei Alexandrescu <andrei@erdani.com> changed:

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

--
June 26, 2017
https://issues.dlang.org/show_bug.cgi?id=16993

Vladimir Panteleev <dlang-bugzilla@thecybershadow.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|dlang.org                   |phobos
           Assignee|nobody@puremagic.com        |issues.dlang@jmdavisProg.co
                   |                            |m

--
July 11, 2017
https://issues.dlang.org/show_bug.cgi?id=16993

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

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

--- Comment #1 from RazvanN <razvan.nitu1305@gmail.com> ---
They don't differ at all. toString just calls toSimpleString and forwards the result.

--
July 11, 2017
https://issues.dlang.org/show_bug.cgi?id=16993

--- Comment #2 from Jonathan M Davis <issues.dlang@jmdavisProg.com> ---
toString does currently call toSimpleString, but the intention is that if you actually care about the format of the string, you don't use toString. Instead, you use a function like toSimpleString or toISOExtString that's explicit about the format. toString is just intended to convert the type to a user-friendly string, not for specific formatting.

--
July 11, 2017
https://issues.dlang.org/show_bug.cgi?id=16993

--- Comment #3 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/57ef9199632b8cb5451f9d7d8c76aad29410db84 Fix issue 16993: Clarify documentation of std.datetime's toString functions.

I should have done this years ago, but this makes the documentation clear that std.datetime's toString is intended simply for easy printing of the type rather than for code that actually cares about the format of the string. There are other, explicit functions for code that actually cares.

https://github.com/dlang/phobos/commit/3324796a2972a2f62ac3f1b73727a5d9e3d5e011 Merge pull request #5592 from jmdavis/issue_16993

Fix issue 16993: Clarify documentation of std.datetime's toString functions. merged-on-behalf-of: Sebastian Wilzbach <sebi.wilzbach@gmail.com>

--
July 11, 2017
https://issues.dlang.org/show_bug.cgi?id=16993

github-bugzilla@puremagic.com changed:

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

--
August 16, 2017
https://issues.dlang.org/show_bug.cgi?id=16993

--- Comment #4 from github-bugzilla@puremagic.com ---
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/57ef9199632b8cb5451f9d7d8c76aad29410db84 Fix issue 16993: Clarify documentation of std.datetime's toString functions.

https://github.com/dlang/phobos/commit/3324796a2972a2f62ac3f1b73727a5d9e3d5e011 Merge pull request #5592 from jmdavis/issue_16993

--
January 05, 2018
https://issues.dlang.org/show_bug.cgi?id=16993

--- Comment #5 from github-bugzilla@puremagic.com ---
Commits pushed to dmd-cxx at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/57ef9199632b8cb5451f9d7d8c76aad29410db84 Fix issue 16993: Clarify documentation of std.datetime's toString functions.

https://github.com/dlang/phobos/commit/3324796a2972a2f62ac3f1b73727a5d9e3d5e011 Merge pull request #5592 from jmdavis/issue_16993

--