Add opDollar aliases for std.range and std.algorithm. #365

Merged
merged 1 commit into from Feb 21, 2012

Conversation

Projects
None yet
3 participants
@dsimcha
Collaborator

dsimcha commented Dec 11, 2011

No description provided.

@andralex

This comment has been minimized.

Show comment
Hide comment
@andralex

andralex Dec 11, 2011

Member

This is exactly the kind of boilerplate that we'd like to avoid. Let me talk to Walter about fixing that in a different way.

Member

andralex commented Dec 11, 2011

This is exactly the kind of boilerplate that we'd like to avoid. Let me talk to Walter about fixing that in a different way.

@dsimcha

This comment has been minimized.

Show comment
Hide comment
@dsimcha

dsimcha Dec 11, 2011

Collaborator

On 12/11/2011 3:29 PM, Andrei Alexandrescu wrote:

This is exactly the kind of boilerplate that we'd like to avoid. Let me talk to Walter about fixing that in a different way.


Reply to this email directly or view it on GitHub:
D-Programming-Language#365 (comment)

100% agreed. I proposed that opDollar automatically forward to length
if no explicit opDollar is present, but that didn't gain any traction.
I just assumed noone else thought the boilerplate was a big deal.

Collaborator

dsimcha commented Dec 11, 2011

On 12/11/2011 3:29 PM, Andrei Alexandrescu wrote:

This is exactly the kind of boilerplate that we'd like to avoid. Let me talk to Walter about fixing that in a different way.


Reply to this email directly or view it on GitHub:
D-Programming-Language#365 (comment)

100% agreed. I proposed that opDollar automatically forward to length
if no explicit opDollar is present, but that didn't gain any traction.
I just assumed noone else thought the boilerplate was a big deal.

@MartinNowak

This comment has been minimized.

Show comment
Hide comment
@MartinNowak

MartinNowak Jan 20, 2012

Member

Please decide what to do with this pull request.
Consider filing an enhancement request and closing it.

Member

MartinNowak commented Jan 20, 2012

Please decide what to do with this pull request.
Consider filing an enhancement request and closing it.

@andralex

This comment has been minimized.

Show comment
Hide comment
@andralex

andralex Jan 20, 2012

Member

I'll discuss it with Walter.

Member

andralex commented Jan 20, 2012

I'll discuss it with Walter.

@dsimcha

This comment has been minimized.

Show comment
Hide comment
@dsimcha

dsimcha Jan 21, 2012

Collaborator

So we're suggesting that UFCS include operators, not just named methods? Interesting.

Collaborator

dsimcha commented Jan 21, 2012

So we're suggesting that UFCS include operators, not just named methods? Interesting.

@andralex

This comment has been minimized.

Show comment
Hide comment
@andralex

andralex Feb 21, 2012

Member

I think I'll merge this in and defer looking into a simpler solution.

Member

andralex commented Feb 21, 2012

I think I'll merge this in and defer looking into a simpler solution.

andralex added a commit that referenced this pull request Feb 21, 2012

Merge pull request #365 from dsimcha/opDollar
Add opDollar aliases for std.range and std.algorithm.

@andralex andralex merged commit 91c4363 into dlang:master Feb 21, 2012

@schveiguy schveiguy referenced this pull request in schveiguy/phobos Apr 15, 2012

Merged

std.process ready for review? #1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment