Thread overview
[Issue 24288] std time handling SUX
Dec 20, 2023
Nick Treleaven
Dec 21, 2023
Dlang Bot
Dec 29, 2023
Dlang Bot
December 20, 2023
https://issues.dlang.org/show_bug.cgi?id=24288

Nick Treleaven <nick@geany.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nick@geany.org

--- Comment #1 from Nick Treleaven <nick@geany.org> ---
This is what I came up with, not sure if there's a better way:

    const tod = cast(TimeOfDay) Clock.currTime;
    (tod.hour * 60 + tod.minute).writeln();

--
December 20, 2023
https://issues.dlang.org/show_bug.cgi?id=24288

--- Comment #2 from crazymonkyyy@gmail.com ---
im pretty sure the intended way is
```d
int min=Clock.currTime.minute;
int hour=Clock.currTime.hour;
```
but to find that you need to read docs across 3 different pages, and good luck
finding the words I was reading given that its all generated docs

and that api isn't great anyway

Its so much code, it needs someone who will delete 95% of the lines

--
December 21, 2023
https://issues.dlang.org/show_bug.cgi?id=24288

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
@ntrel created dlang/phobos pull request #8867 "[std.datetime] Tweak docs" mentioning this issue:

- [std.datetime] Tweak docs

  Add links to package.d.
  Tweak formatting for SysTime, use PANEL.
  Use REF_SHORT after first reference.
  List TimeOfDay cast target.
  Move currTime link to See Also.
  Fix anchor for currTime - need `.` prefix before fully-qualified names.

  Part of Issue 24288.

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

--
December 29, 2023
https://issues.dlang.org/show_bug.cgi?id=24288

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/phobos pull request #8867 "[std.datetime] Tweak docs" was merged into master:

- 5e93d4f9c9f0a17fdf6f222c9ff8153b16aa01c1 by Nick Treleaven:
  [std.datetime] Tweak docs

  Add links to package.d.
  Tweak formatting for SysTime, use PANEL.
  Use REF_SHORT after first reference.
  List TimeOfDay cast target.
  Move currTime link to See Also.
  Fix anchor for currTime - need `.` prefix before fully-qualified names.

  Part of Issue 24288.

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

--
October 18
https://issues.dlang.org/show_bug.cgi?id=24288

Adam Wilson <flyboynw@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |flyboynw@gmail.com
         Resolution|---                         |FIXED

--- Comment #5 from Adam Wilson <flyboynw@gmail.com> ---
Their was a PR for this but the bot didn't resolve it. Resolving manually.

--
December 01
https://issues.dlang.org/show_bug.cgi?id=24288

--- Comment #6 from dlangBugzillaToGithub <robert.schadek@posteo.de> ---
THIS ISSUE HAS BEEN MOVED TO GITHUB

https://github.com/dlang/phobos/issues/10539

DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB

--