[Issue 16992] fromISOString, fromISOExtString, and fromSimpleString do not have examples

via Digitalmars-d-bugs digitalmars-d-bugs at puremagic.com
Tue Jul 4 21:19:22 PDT 2017


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

Jonathan M Davis <issues.dlang at jmdavisProg.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |issues.dlang at jmdavisProg.co
                   |                            |m

--- Comment #8 from Jonathan M Davis <issues.dlang at jmdavisProg.com> ---
(In reply to RazvanN from comment #7)
> I think that having all fromISOString overloads gathered in a single
> template function which is called fromISOString!Choose_type(some_string),
> where Choose_type is one of {Date, DateTime, TimeOfDay} is more elegant and
> the documentation can be presented a single time; rather than having 3
> fromISOString functions dispersed through the documentation.

They're dispersed, because they go with each individual type and are specific
to each individual type. They have distinct implemnetations and should be
documented as such. I see no reason to have a module-level wrapper function -
especially when not all of the time point types are even in the same module
anymore, now that std.datetime has been split up.

The issue here is that the documentation examples do not show up in the
ddoc-generated documentation in spite of the fact that they are in the code as
ddoc-ed examples. There is no need to change std.datetime here. We need to fix
the compiler.

--


More information about the Digitalmars-d-bugs mailing list