Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Cor v2 feedback #11

Closed
Ovid opened this issue May 12, 2020 · 5 comments
Closed

Cor v2 feedback #11

Ovid opened this issue May 12, 2020 · 5 comments
Labels
Feedback Cor Proposal Feedback

Comments

@Ovid
Copy link
Collaborator

Ovid commented May 12, 2020

Please leave comments on Cor v2 here.

@Ovid Ovid added the Feedback Cor Proposal Feedback label May 12, 2020
@scopesoft
Copy link

Having read the Cor documentation, when wanting to use Cor it would come down to performance. The reason I say this is that I have the choice of using Cor or Moo/Moose. The syntax of Cor is on the cryptic side when compared to Moo/Moose. And thus I would choose Moo/Moose over Cor. So as I said it would come down to performance, which I think was stated somewhere else would not be a priority on initial release.

@Ovid
Copy link
Collaborator Author

Ovid commented Jul 13, 2020

@scopesoft I still recall when I first used Moose and found it painful. Like any large, new "thing", it takes time to learn. For Cor, slot attributes are mostly self-documenting and they're composable, so I suspect that will flatten the learning curve.

As for performance, we expect it will improve dramatically because internally, there's no method call overhead to get slot data. Paul Evan's Object::Pad, while having a relatively unoptimized constructor, nonetheless is showing that its performance appears to be faster than core Perl (not just Moose) in terms of accessing that data. And that's with very little work done to optimize it. I doubt that performance will be a serious issue.

@duncand
Copy link

duncand commented Jun 13, 2021

Under "Try/Catch/Finally" for v2 it says this really needs to be in core. That can probably be removed or updated to say that actually WAS added to core in 5.34.

@Ovid
Copy link
Collaborator Author

Ovid commented Jun 14, 2021

@duncand Removed try/catch note. Thanks!

@Ovid
Copy link
Collaborator Author

Ovid commented Aug 15, 2021

This repository is currently shifting its focus to the MVP. This is resolved for the MVP. Further issues should be new tickets.

@Ovid Ovid closed this as completed Aug 15, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Feedback Cor Proposal Feedback
Projects
None yet
Development

No branches or pull requests

3 participants