Thread overview
[Issue 18374] Add range functions to Nullable
Jun 15, 2018
Alex
Jun 24, 2018
Seb
Jun 25, 2018
Mitu
June 15, 2018
https://issues.dlang.org/show_bug.cgi?id=18374

Alex <sascha.orlov@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sascha.orlov@gmail.com

--
June 24, 2018
https://issues.dlang.org/show_bug.cgi?id=18374

Seb <greensunny12@gmail.com> changed:

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

--- Comment #1 from Seb <greensunny12@gmail.com> ---
Are you aware of the new `apply`?

https://dlang.org/changelog/2.080.0.html#std-typecons-nullable-apply

It still would be great to have Nullable and ranges working nicely together, but at least apply is a start.

--
June 25, 2018
https://issues.dlang.org/show_bug.cgi?id=18374

--- Comment #2 from Mitu <the.mail.of.mi2@gmail.com> ---
(In reply to Seb from comment #1)
> Are you aware of the new `apply`?
> 
> https://dlang.org/changelog/2.080.0.html#std-typecons-nullable-apply
> 
> It still would be great to have Nullable and ranges working nicely together, but at least apply is a start.

I am, but it wasn't there the day I have created this issue.

apply() definitely does the trick. It still lacks one case though - when we want to call the void function unless the value is null:

---------------
Nullable!int something;
something.apply!writeln;
---------------

I still think that Nullable as a range might be more powerful and its integration with the range behavior might be save some LoC in some places, but I cannot come up of an example now.

--
March 27, 2022
https://issues.dlang.org/show_bug.cgi?id=18374

elpenguino+D@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |elpenguino+D@gmail.com
         Resolution|---                         |FIXED

--- Comment #3 from elpenguino+D@gmail.com ---
Fixed by https://github.com/dlang/phobos/pull/8417

--