Thread overview
[Issue 19082] Cannot inline "...Slides.numberOfFullFrames", "...Slides.gap"
Jul 14, 2018
Seb
Jul 16, 2018
Mitu
Jul 16, 2018
Mitu
Jul 16, 2018
Seb
Jul 17, 2018
Mitu
Jul 17, 2018
Seb
Jul 17, 2018
Seb
July 14, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

Seb <greensunny12@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |greensunny12@gmail.com

--- Comment #1 from Seb <greensunny12@gmail.com> ---
We would need a minimal reproducible example to help you :/

--
July 16, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

--- Comment #2 from Mitu <the.mail.of.mi2@gmail.com> ---
OK, I managed to find a minimal example. The code below builds properly for debug build, but fails with the release build.

=================================
/+ dub.sdl: name "main" +/

import std.algorithm;
import std.range;

void main()
{
[1].map!(x => x).slide(2);
}
===========

Replacing .map!(x => x) with .map!"a" makes the code code work strangely.

--
July 16, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

--- Comment #3 from Mitu <the.mail.of.mi2@gmail.com> ---
OK, I managed to find a minimal example. The code below builds properly for debug build, but fails with the release build.

==================================
/+ dub.sdl: name "main" +/

import std.algorithm;
import std.range;

void main()
{
        [1].map!(x => x).slide(2);
}
==================================

Replacing .map!(x => x) with .map!"a" makes the code code work strangely.

--
July 16, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

Seb <greeenify@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |greeenify@gmail.com

--- Comment #4 from Seb <greeenify@gmail.com> ---
I'm sorry, but I can't reproduce this:

https://run.dlang.io/is/W8BEnJ https://run.dlang.io/is/mHxy3W

Have you tried DustMite?

--
July 17, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

--- Comment #5 from Mitu <the.mail.of.mi2@gmail.com> ---
(In reply to Seb from comment #4)
> I'm sorry, but I can't reproduce this:
> 
> https://run.dlang.io/is/W8BEnJ https://run.dlang.io/is/mHxy3W
> 
> Have you tried DustMite?

You seem to have forgotten that dub release build insterts -inline. Try this:

https://run.dlang.io/is/acZVMa

--
July 17, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

--- Comment #6 from Seb <greeenify@gmail.com> ---
Oh I see.
This should fix it: https://github.com/dlang/phobos/pull/6636

--
July 17, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

Seb <greeenify@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|enhancement                 |major

--
August 07, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

--- Comment #7 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/fc2a1d0e110d3dd422d4e10262b74774fa93fcc1 Fix Issue 19082 - Cannot inline "...Slides.numberOfFullFrames", "...Slides.gap"

https://github.com/dlang/phobos/commit/a6e53eaa98fe539acdefb7d3cfed982bb0fc2bed Merge pull request #6636 from wilzbach/fix-19082

Fix Issue 19082 - Cannot inline "...Slides.numberOfFullFrames", "...Slides.gap"

--
August 07, 2018
https://issues.dlang.org/show_bug.cgi?id=19082

github-bugzilla@puremagic.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--