[Issue 12293] forward should be moved to std.functional
https://issues.dlang.org/show_bug.cgi?id=12293 Andrej Mitrovic andrej.mitrov...@gmail.com changed: What|Removed |Added Summary|std.algorithm.forward |forward should be moved to |documentation |std.functional Severity|trivial |normal --
[Issue 12293] forward should be moved to std.functional
https://issues.dlang.org/show_bug.cgi?id=12293 --- Comment #1 from Kenji Hara k.hara...@gmail.com --- (In reply to Gary Willoughby from comment #0) std.algorithm.forward is missing from the index at the top of http://dlang.org/phobos/std_algorithm.html If the issue is the un-listing of 'forward' function, why it should go std.functional module? You should explain the reason. --
[Issue 12293] forward should be moved to std.functional
https://issues.dlang.org/show_bug.cgi?id=12293 --- Comment #2 from Andrej Mitrovic andrej.mitrov...@gmail.com --- (In reply to Kenji Hara from comment #1) (In reply to Gary Willoughby from comment #0) std.algorithm.forward is missing from the index at the top of http://dlang.org/phobos/std_algorithm.html If the issue is the un-listing of 'forward' function, why it should go std.functional module? You should explain the reason. No, I've changed the title. See the history here: https://issues.dlang.org/show_activity.cgi?id=12293 --
[Issue 12293] forward should be moved to std.functional
https://issues.dlang.org/show_bug.cgi?id=12293 --- Comment #3 from Kenji Hara k.hara...@gmail.com --- (In reply to Andrej Mitrovic from comment #2) No, I've changed the title. See the history here: https://issues.dlang.org/show_activity.cgi?id=12293 If the reason comes from just your feeling, I cannot agree with the title. std.algorithm.forward is in the same category of std.algorithm.move and swap. Why they are in std.algorithm module? Because in C++03, std::swap was in algorithm header. So in D, swap, move, and forward are in std.algorithm. I know that std::swap, move, and forward are in utility header in C++11. But, std.functional module is for functional programming utilities. From the perspective, swap, move, and forward are not directly related to functional programming IMO. So, I think that moving it to the module is unreasonable change. --
[Issue 12293] forward should be moved to std.functional
https://issues.dlang.org/show_bug.cgi?id=12293 --- Comment #4 from Andrej Mitrovic andrej.mitrov...@gmail.com --- (In reply to Kenji Hara from comment #3) (In reply to Andrej Mitrovic from comment #2) No, I've changed the title. See the history here: https://issues.dlang.org/show_activity.cgi?id=12293 If the reason comes from just your feeling, I cannot agree with the title. I wasn't the one to suggest it, it was monarchdodra. Originally I made the pull that added 'forward' to the cheat-sheet of the std.algorithm page. I can recreate this pull again if desired. std.algorithm.forward is in the same category of std.algorithm.move and swap. Why they are in std.algorithm module? Because in C++03, std::swap was in algorithm header. So in D, swap, move, and forward are in std.algorithm. That sounds reasonable to me then. Unless monarchdodra objects I'll recreate the cheat-sheet pull. --
[Issue 12293] forward should be moved to std.functional
https://issues.dlang.org/show_bug.cgi?id=12293 --- Comment #5 from monarchdo...@gmail.com --- (In reply to Andrej Mitrovic from comment #4) (In reply to Kenji Hara from comment #3) (In reply to Andrej Mitrovic from comment #2) No, I've changed the title. See the history here: https://issues.dlang.org/show_activity.cgi?id=12293 If the reason comes from just your feeling, I cannot agree with the title. I wasn't the one to suggest it, it was monarchdodra. What !? AndrejMitrovic commented 4 days ago https://issues.dlang.org/show_bug.cgi?id=12293 Honestly I think forward doesn't belong in std.algorithm. It should likely belong to std.functional or a similar place. Hence why I had to add a utility category. I'd rather we just move the template out of std.algorithm though. You suggested it. I just told you to go through with it. Originally I made the pull that added 'forward' to the cheat-sheet of the std.algorithm page. I can recreate this pull again if desired. Sure. std.algorithm.forward is in the same category of std.algorithm.move and swap. Why they are in std.algorithm module? Because in C++03, std::swap was in algorithm header. So in D, swap, move, and forward are in std.algorithm. That sounds reasonable to me then. Unless monarchdodra objects I'll recreate the cheat-sheet pull. Sounds reasonable to me too. Revert back to how it was? --
[Issue 12293] forward should be moved to std.functional
https://issues.dlang.org/show_bug.cgi?id=12293 --- Comment #6 from Andrej Mitrovic andrej.mitrov...@gmail.com --- (In reply to monarchdodra from comment #5) What !? You suggested it. I just told you to go through with it. *Facepalm*. It appears I did. Sorry for this. : --