Method::Signature lexing support #6

Merged
merged 3 commits into from Mar 19, 2014

Conversation

Projects
None yet
3 participants
@AkbarTheGreat

Method::Signatures (and MooseX::Method::Signatures) both ostensibly add keywords to the perl language, func and method. While these add a great deal of utility for developers, it can be frustrating to use EPIC with a standard perl-only lexing scheme as that keeps outlines and other nice features of the editor from working properly. The fix I have proposed here adds an option to the preferences pane to enable/disable this departure from Perl-only lexing, and adds the ability for the lexer to pull out func & method as if they were sub.

Tracy Beck added some commits Aug 27, 2013

Tracy Beck
Adding functionality for Method::Signatures
If you use Method::Signatures, it adds keywords, func & method, which behave like sub
Doing so causes the lexer to fail to recognize these as subroutines.
Now, they can be treated as equivalent to "sub" iff the user opts into that functionality
@jploski

This comment has been minimized.

Show comment Hide comment
@jploski

jploski Aug 28, 2013

Can you please provide a pull request without the noise caused by trimming of trailing whitespace in source?

Can you please provide a pull request without the noise caused by trimming of trailing whitespace in source?

This comment has been minimized.

Show comment Hide comment
@AkbarTheGreat

AkbarTheGreat Aug 28, 2013

Owner

Sure, I can do that. Sorry for the inconvenience, I have AnyEdit set to do that on save, I should've turned it off before working on your codebase.

Owner

AkbarTheGreat replied Aug 28, 2013

Sure, I can do that. Sorry for the inconvenience, I have AnyEdit set to do that on save, I should've turned it off before working on your codebase.

@Jasonra

This comment has been minimized.

Show comment Hide comment
@Jasonra

Jasonra Oct 15, 2013

Ah, exactly what I was looking for! Can't wait for this to be in the official release!

Jasonra commented Oct 15, 2013

Ah, exactly what I was looking for! Can't wait for this to be in the official release!

@AkbarTheGreat

This comment has been minimized.

Show comment Hide comment
@AkbarTheGreat

AkbarTheGreat Mar 19, 2014

Jan,

Have you had a chance to look at this fix?

--Tracy

Jan,

Have you had a chance to look at this fix?

--Tracy

jploski added a commit that referenced this pull request Mar 19, 2014

Merge pull request #6 from AkbarTheGreat/testing
Method::Signature lexing support

@jploski jploski merged commit 4c5721a into jploski:testing Mar 19, 2014

@jploski

This comment has been minimized.

Show comment Hide comment
@jploski

jploski Mar 19, 2014

Owner

Thanks for the contribution and sorry, that got off my radar completely. Now merged, I'm going to release a new version of 'testing' over this weekend.

Owner

jploski commented Mar 19, 2014

Thanks for the contribution and sorry, that got off my radar completely. Now merged, I'm going to release a new version of 'testing' over this weekend.

@otrosien otrosien referenced this pull request in otrosien/epic-ide-issues Jun 7, 2017

Closed

"method" keyword should be treated the same as "sub" #895

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