Pull requests

#181 Merged
Repository
hoelzro hoelzro
Branch
default
Repository
birkenfeld birkenfeld
Branch
default

Add Perl 6 lexer

Author
  1. Rob Hoelz
Reviewers
Description

Hi there,

Over the past few months, I have been working on a pygments lexer for Perl 6. The lexer is quite dense, so I understand if it takes some time to review. Please don't hesitate to ask if you have any questions or concerns about my code.

Thanks, Rob

Comments (8)

  1. Tim Hatch

    Looking good. I need to manually check the lines with (?P= in them, regexlint flagged them as being unusual. For the example files, two things: could you name them .pm6 so "make test" works, and are the source files under an appropriate license to be included in BSD source? I couldn't find their source in a couple minutes of searching, is it something you wrote or do we need to find an author to seek appropriate relicensing? (Evidently Artistic 2.0 code can be included like this, but I'd rather go with something simpler as an examplefile that we can clearly say is BSD, if necessary.)

  2. Rob Hoelz author

    I've renamed the .pm files to .pm6 (I can't get make test to work myself; the test suite seems to take issue with Python 3 being /usr/bin/python). Grammar.pm6 and Optimizer.pm6 are part of the Rakudo codebase under the Artistic 2.0 license; everything else was written by me as examples. We don't need to keep those two files; they just happened to be good examples for testing.

  3. Tim Hatch

    Sorry for the delay -- looks good. Can you remove the files from Rakudo, and then I'll merge?

    (An aside for Georg -- we need to rearchitect guess_lexer and friends, until we do, guessing for .pl may be weird.)