← Back to index | Original Bugzilla link

Bug 16948 – broken links in std.stdio due to inccorrect use of WEB macro

Status
RESOLVED
Resolution
FIXED
Severity
minor
Priority
P1
Component
dlang.org
Product
D
Version
D2
Platform
All
OS
All
Creation time
2016-12-04T13:58:00Z
Last change time
2017-01-16T23:25:20Z
Keywords
ddoc
Assigned to
ag0aep6g
Creator
lucien.perregaux

Comments

Comment #0 by lucien.perregaux — 2016-12-04T13:58:18Z
Hello, When parsing a $(WEB https://..., bug X), the "https" doesn't override the "http". Explanation: --------------------------------------- this line: https://github.com/dlang/phobos/blob/master/std/stdio.d#L4375 produce: <a href="http://https://issues.dlang.org/show_bug.cgi?id=15768">bug 15768</a> instead of: <a href="https://issues.dlang.org/show_bug.cgi?id=15768">bug 15768</a> --------------------------------------- That's all !
Comment #1 by ag0aep6g — 2016-12-04T21:13:42Z
I'm assuming that you just want the broken links to be fixed. Pull request for that: https://github.com/dlang/phobos/pull/4931 I'm changing the title of this issue to reflect that it's about the links. If you're actually asking for an improved WEB macro, please clarify. Maybe file a separate issue then.
Comment #2 by lucien.perregaux — 2016-12-05T12:06:15Z
Thanks for the fix.
Comment #3 by github-bugzilla — 2016-12-05T16:35:47Z
Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/c8a123f4d0f2cc9b611545f5c3055cfcc31fcb05 fix issue 16948 - broken links in std.stdio due to inccorrect use of WEB macro https://github.com/dlang/phobos/commit/68cf6e8d1e6d8a1b43bff65fcbaef324bd158497 Merge pull request #4931 from aG0aep6G/16948 fix issue 16948 - broken links in std.stdio due to inccorrect use of WEB macro
Comment #4 by github-bugzilla — 2016-12-07T16:09:13Z
Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/c8a123f4d0f2cc9b611545f5c3055cfcc31fcb05 fix issue 16948 - broken links in std.stdio due to inccorrect use of WEB macro https://github.com/dlang/phobos/commit/68cf6e8d1e6d8a1b43bff65fcbaef324bd158497 Merge pull request #4931 from aG0aep6G/16948
Comment #5 by github-bugzilla — 2017-01-16T23:25:20Z
Commits pushed to newCTFE at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/c8a123f4d0f2cc9b611545f5c3055cfcc31fcb05 fix issue 16948 - broken links in std.stdio due to inccorrect use of WEB macro https://github.com/dlang/phobos/commit/68cf6e8d1e6d8a1b43bff65fcbaef324bd158497 Merge pull request #4931 from aG0aep6G/16948