p5-mop-redux icon indicating copy to clipboard operation
p5-mop-redux copied to clipboard

Improvements for overloading

Open tobyink opened this issue 12 years ago • 5 comments

  • Test cases for overloading, including a TODO (for which I'll file a separate issue).
  • Allow attributes to take part in overloading without needing an accessor. This only makes sense for unary-like overloads.
class Person {
       has $name is overload(q[""]);
}

tobyink avatar Jul 20 '13 20:07 tobyink

Can you rebase this one too?

doy avatar Aug 09 '13 16:08 doy

I can't do it easily (assuming you are talking to me)

On Aug 9, 2013, at 12:17 PM, Jesse Luehrs [email protected] wrote:

Can you rebase this one too?

— Reply to this email directly or view it on GitHub.

stevan avatar Aug 09 '13 16:08 stevan

I was talking to @tobyink, as it's his pull request(:

doy avatar Aug 09 '13 16:08 doy

Actually, I am really not a fan of the overload on attributes.

On Aug 9, 2013, at 12:43 PM, Jesse Luehrs [email protected] wrote:

I was talking to @tobyink, as it's his pull request(:

— Reply to this email directly or view it on GitHub.

stevan avatar Aug 09 '13 16:08 stevan

Actually, I am okay with this if we explicitly check for unary operators and die on all others.

A test case for ++ and -- would be nice as those are two operators that would maybe make sense for this.

Also maybe %{} and family as well.

On Aug 9, 2013, at 12:45 PM, Stevan Little [email protected] wrote:

Actually, I am really not a fan of the overload on attributes.

On Aug 9, 2013, at 12:43 PM, Jesse Luehrs [email protected] wrote:

I was talking to @tobyink, as it's his pull request(:

— Reply to this email directly or view it on GitHub.

stevan avatar Aug 09 '13 16:08 stevan