Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Remove spurious "next" in lexicon_get_glob() #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Remove spurious "next" in lexicon_get_glob() #20

wants to merge 1 commit into from

Commits on Feb 28, 2018

  1. Remove spurious "next" in lexicon_get_glob() 

    When faced with a *DATA file glob, lexicon_get_glob()
will:

    1 Walk the call stack, looking for the package that
      contains that *DATA glob.
    2 Open that module's source file.
    3 Skip up to and including the __DATA__ line.
    4 Read the rest of the input as the content to parse.

... Except that step 1 doesn't work. Due to an unqualified "next"
statement, it was just basically running an empty loop.

The result was that the below would just never work:

    use Locale::Maketext::Lexicon( Gettext => \*DATA );

This made me sad.
    @sbakker
    sbakker committed Feb 28, 2018
    Configuration menu
    Copy the full SHA
    ad90d70 View commit details
    Browse the repository at this point in the history