#1819: mk_language_shell.pl generates broken PMC boilerplate
--------------------+-------------------------------------------------------
Reporter: ellefu | Owner: dukeleto
Type: patch | Status: new
Priority: normal | Milestone:
Component: tools | Version: 2.8.0
Severity: medium | Keywords:
Lang: | Patch: new
Platform: unix |
--------------------+-------------------------------------------------------
Comment(by cotto):
ellefu, I applied (but didn't commit) your patch (updated_patch.txt), and
while it does fix some obvious problems, the generated PMC still doesn't
build. You're welcome to get the PMC to build properly (and I'll be glad
to apply such a patch).
If you're feeling more ambitious, I think it'd be helpful if new HLL
developers had a more user-friendly PMC as a template. Looking at the
generated PMC, it's not entirely clear what it is or what it's supposed to
do. When a new developer comes along, it'd be better to present a PMC
with common semantics (i.e. Integer, Hash, etc) so that the developer
won't have to spend as much time figuring out what the PMC does before
moving on to figuring out how it does it. That said, I'm sure any new
developer will spend lots of time looking at Parrot's core PMCs and any
dynpmcs available, so there's no need for the template PMC to be anything
more than a good clear starting point.
--
Ticket URL: <https://trac.parrot.org/parrot/ticket/1819#comment:4>
Parrot <https://trac.parrot.org/parrot/>
Parrot Development
_______________________________________________
parrot-tickets mailing list
[email protected]
http://lists.parrot.org/mailman/listinfo/parrot-tickets
