#12 sub detection broken (slightly)

closed-fixed
Outline (28)
5
2004-05-12
2003-05-13
No

Subroutine detection for the Outline view fails if the
subroutine is defined on one line, and followed by a
semicolon, eg.

sub version { '$Id $' };

Now, I agree that's not good code - the semi is not
needed. (And not something I'm likely to leave now
that it's been pointed out to me :) But it is _valid_
code according to Perl, so shouldn't break the EPIC parser.

Discussion

  • Jochen Ruehl

    Jochen Ruehl - 2003-07-27
    • assigned_to: nobody --> luelljoc
     
  • Lee Wadwell

    Lee Wadwell - 2004-01-30

    Logged In: YES
    user_id=963829

    Another variation of this bug (and more annoying, well at
    least for me) is when an "optional" prototype is used in the
    subroutine declaration.
    Same issue...there is a semicolon on the declaration line, but
    its more than a 'one liner'...eg

    sub mySub ($$;$) {
    ...
    }

    In this case, the semi is needed and it is good code. Yes I
    can move the prototypes to the next line and everything is
    fine...except its not what I would call 'good style'.

     
  • Jochen Ruehl

    Jochen Ruehl - 2004-05-12
    • labels: --> Outline
    • status: open --> closed-fixed
     
  • Jochen Ruehl

    Jochen Ruehl - 2004-05-12

    Logged In: YES
    user_id=274815

    Should be fixed in EPIC 0.1.0.
    If not please reopen the bug.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks