March 21, 2020
https://issues.dlang.org/show_bug.cgi?id=17712

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|b2.temp@gmx.com             |

--
June 19, 2020
https://issues.dlang.org/show_bug.cgi?id=17712

Nick Treleaven <nick@geany.org> changed:

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

--- Comment #19 from Nick Treleaven <nick@geany.org> ---
(In reply to kinke from comment #17)
> I'm not too happy about closing this, as the underlying issue definitely isn't fixed.

Yes, and the workaround didn't even add a test case to prevent someone just changing std.conv.text to use toChars again. The toChars radix workaround (comment #11) would have had much lower impact. I haven't been able to reproduce the original bug with dmd v2.092.0, maybe I'm doing something wrong though.

--
June 19, 2020
https://issues.dlang.org/show_bug.cgi?id=17712

Steven Schveighoffer <schveiguy@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://issues.dlang.org/sh
                   |                            |ow_bug.cgi?id=20950

--
June 19, 2020
https://issues.dlang.org/show_bug.cgi?id=17712

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #20 from Dlang Bot <dlang-bot@dlang.rocks> ---
@ntrel created dlang/dmd pull request #11300 "Fix Issue #17712 - Add test cases" fixing this issue:

- Fix Issue #17712 - Add test cases

https://github.com/dlang/dmd/pull/11300

--
June 22, 2020
https://issues.dlang.org/show_bug.cgi?id=17712

moonlightsentinel@disroot.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |moonlightsentinel@disroot.o
                   |                            |rg

--- Comment #21 from moonlightsentinel@disroot.org ---
Reduced test case without phobos that still fails with current dmd:

=============================================
// stdx/conv.d:

void to(T)(uint args)
{
    toChars!10(args);
}

alias CodepointSet = InversionList!();

struct InversionList()
{
    void toSourceCode()
    {
        toChars(0u);
    }
}

void toChars(ubyte radix = 10, T)(T) {}

=============================================
// test.d:

void main()
{
    import stdx.conv;
    to!string(0u);
    CodepointSet().toSourceCode();
}
=============================================

dmd -c stdx/conv.d
dmd test.d conv.o

--
April 10, 2023
https://issues.dlang.org/show_bug.cgi?id=17712

RazvanN <razvan.nitu1305@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
                 CC|                            |razvan.nitu1305@gmail.com
         Resolution|---                         |WORKSFORME

--- Comment #22 from RazvanN <razvan.nitu1305@gmail.com> ---
(In reply to moonlightsentinel from comment #21)
> Reduced test case without phobos that still fails with current dmd:
> 
> =============================================
> // stdx/conv.d:
> 
> void to(T)(uint args)
> {
>     toChars!10(args);
> }
> 
> alias CodepointSet = InversionList!();
> 
> struct InversionList()
> {
>     void toSourceCode()
>     {
>         toChars(0u);
>     }
> }
> 
> void toChars(ubyte radix = 10, T)(T) {}
> 
> =============================================
> // test.d:
> 
> void main()
> {
>     import stdx.conv;
>     to!string(0u);
>     CodepointSet().toSourceCode();
> }
> =============================================
> 
> dmd -c stdx/conv.d
> dmd test.d conv.o

I cannot reproduce this.

--
1 2 3
Next ›   Last »