Bug 16191 – std/digest/digest.d should be renamed to package.d

Status
RESOLVED
Resolution
FIXED
Severity
minor
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2016-06-21T20:23:20Z
Last change time
2018-01-05T13:30:08Z
Assigned to
No Owner
Creator
Steven Schveighoffer
See also
https://issues.dlang.org/show_bug.cgi?id=17724

Comments

Comment #0 by schveiguy — 2016-06-21T20:23:20Z
The fully qualified name of the digest function template (https://dlang.org/phobos/std_digest_digest.html#digest) is std.digest.digest.digest. This is because std.digest is a package, with a module named digest in it, and the function digest inside that. I think it's reasonable to have std.digest.digest. Even though there's repetition, it's clear that the first digest is a module, the second is a function name. But there's no reason (especially with the advent of package.d) to have a std.digest.digest module.
Comment #1 by greeenify — 2017-01-04T16:57:20Z
Comment #2 by github-bugzilla — 2017-07-10T03:50:16Z
Commit pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/5774d017eb4fa154491385a91f590abc2f66b964 Fix issue 16191 - std/digest/digest.d should be renamed to package.d
Comment #3 by github-bugzilla — 2017-08-16T13:23:52Z
Commit pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/5774d017eb4fa154491385a91f590abc2f66b964 Fix issue 16191 - std/digest/digest.d should be renamed to package.d
Comment #4 by github-bugzilla — 2018-01-05T13:30:08Z
Commit pushed to dmd-cxx at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/5774d017eb4fa154491385a91f590abc2f66b964 Fix issue 16191 - std/digest/digest.d should be renamed to package.d