Menu

#1016 Symbols list breaks when first line points to node

v1.23.1
closed-fixed
v1.23.1
6
2013-12-24
2013-12-24
Yuzem
No

I'm using Geany v1.23.1 in Ubuntu 13.10

Steps to reproduce the error:

  1. Create a file test.js
  2. Copy / paste the following content:

    #!/usr/bin/env nodejs
    
    function test1 ()
    {
    
    }
    
    function test2 ()
    {
    
    }
    
  3. The symbols list doesn't list the functions.

  4. Remove the first line and the symbols list works correctly.

Discussion

  • Colomban Wendling

    • labels: --> parser, JavaScript
    • status: open --> closed-fixed
    • assigned_to: Colomban Wendling
     
  • Colomban Wendling

    Fixed in [a7884ee2f8b746e70cc2d71cf898a4d5a95172e0], thanks for the report.

     

    Related

    Commit: [a7884e]

    • Yuzem

      Yuzem - 2013-12-24

      You're welcome but just in case, is it ok that in the link you posted it says:
      +#!/usr/bin/end nodejs
      Instead of:
      +#!/usr/bin/env nodejs

       
      • Colomban Wendling

        Oh yeah, that's a typo. Though, it's harmless, anything after the #! is ignored anyway.

         
  • Colomban Wendling

    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -2,22 +2,22 @@
    
     Steps to reproduce the error:
    
    -. Create a file test.js
    -. Copy / paste the following content:
    +1. Create a file test.js
    +2. Copy / paste the following content:
    +
    +        :::js
    +        #!/usr/bin/env nodejs
    +        
    +        function test1 ()
    +        {
    +           
    +        }
    +        
    +        function test2 ()
    +        {
    +           
    +        }
    
    -`#!/usr/bin/env nodejs`
    -
    -function test1 ()
    -{
    -   
    -}
    -
    -function test2 ()
    -{
    -   
    -}
    -
    -
    -. The symbols list doesn't list the functions.
    -. Remove the first line and the symbols list works correctly.
    +3. The symbols list doesn't list the functions.
    +4. Remove the first line and the symbols list works correctly.
    
     

Log in to post a comment.