Bug 11832 – std.datetime: ddoc warnings

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P2
Component
dmd
Product
D
Version
D2
Platform
All
OS
All
Creation time
2013-12-27T14:46:00Z
Last change time
2014-08-18T04:50:08Z
Assigned to
nobody
Creator
andrei

Comments

Comment #0 by andrei — 2013-12-27T14:46:42Z
std/datetime.d(1726): Warning: Ddoc: function declaration has no parameter 'minutes' std/datetime.d(2111): Warning: Ddoc: function declaration has no parameter 'tz' std/datetime.d(8468): Warning: Ddoc: function declaration has no parameter 'isoString' std/datetime.d(10563): Warning: Ddoc: parameter count mismatch std/datetime.d(11227): Warning: Ddoc: function declaration has no parameter 'value' std/datetime.d(14626): Warning: Ddoc: function declaration has no parameter 'isoString' std/datetime.d(16114): Warning: Ddoc: function declaration has no parameter 'value' std/datetime.d(17850): Warning: Ddoc: function declaration has no parameter 'isoString' std/datetime.d(19467): Warning: Ddoc: parameter count mismatch std/datetime.d(22107): Warning: Ddoc: function declaration has no parameter 'dir' std/datetime.d(22107): Warning: Ddoc: parameter count mismatch std/datetime.d(23535): Warning: Ddoc: function declaration has no parameter 'begin' std/datetime.d(24369): Warning: Ddoc: function declaration has no parameter 'dir' std/datetime.d(24369): Warning: Ddoc: parameter count mismatch std/datetime.d(29606): Warning: Ddoc: parameter count mismatch std/datetime.d(31455): Warning: Ddoc: function declaration has no parameter 'sysTime'
Comment #1 by andrei — 2013-12-27T14:47:15Z
these warnings are produced with make -f posix.mak html for phobos
Comment #2 by mailnew4ster — 2013-12-27T14:57:33Z
Comment #3 by issues.dlang — 2013-12-27T16:27:23Z
@9999 Yes, any changes to std.datetime will cause me more merge problems in finishing splitting std.datetime, but if any changes actually need to get in (e.g. a bug fix), then they should be merged (and then I'll just have to deal with the merge conflicts), particularly since I've been swamped at work for the past few months and unable to finish splitting std.datetime (I'm in the middle of reworking the documentation at this point). Right now, I'm expecting to get back to D stuff towards the beginning of February, but I'm effectively a road block until then with regards to any std.datetime changes which rely on me, and while I'd prefer that more changes not be made to std.datetime in the interim and cause me more work, if they need to be done, they need to be done. But I would appreciate it if trivial stuff didn't get changed as it will create more work for me. I'll make sure that all such warnings are fixed as I rework the documentation, but if anyone feels like they need to be fixed before that, then they can create a pull request for it.
Comment #4 by github-bugzilla — 2014-08-16T21:48:58Z
Commit pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/105b2c2d9f31219d8eb8eede0a25d6bc375fa5d0 Merge pull request #2426 from quickfur/issue11832 Issue 11832: Fix mismatching param name.
Comment #5 by hsteoh — 2014-08-18T04:50:08Z
All warnings have been fixed in git HEAD.