Skip to content

Symbol list uses 100% CPU #71

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.

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

Closed
pedro-ricardo opened this issue Jun 7, 2018 · 3 comments
Closed

Symbol list uses 100% CPU #71

pedro-ricardo opened this issue Jun 7, 2018 · 3 comments

Comments

@pedro-ricardo
Copy link
Collaborator

The code below freezes the extension when the Symbol List (Ctrl+Shift+O) command is used. The extension keeps running in the background with 100% CPU even after VScode is closed.

module symbol_test
    
    implicit none
    
contains
    
    subroutine Much_dummy_arguments(headlev, rhs, rho, rhof, mu, u, u1, v, v1, w, w1, sx, sy, sz, &
        kindv, crlx, rlx_down, rlx_up, shift, t, dt, alfa, theta, tol_vel, ap_simple)
        implicit none
        
    end subroutine Much_dummy_arguments
    
end module symbol_test

The screenshot below illustrates what is happening:
symb_err
After pressing Ctrl+Shift+O it is not possible to type anything, the editor freezes.
If you kill the code process that is using 100% CPU it is possible to type in editor once again, but the symbol list shows No symbol information for the file and the following window pops up:
err

@pedro-ricardo
Copy link
Collaborator Author

Error does not happen if user setting"fortran.symbols": [ ] is blank.

@pedro-ricardo
Copy link
Collaborator Author

Most likelly the error was introduced by commit #64

@pedro-ricardo
Copy link
Collaborator Author

Fixed in PR #72

krvajal pushed a commit that referenced this issue Jun 16, 2018
* Fix subroutine regex for more that one line of arguments. Issue #71

* Add word bound '\b' on subroutine regex. Avoid false positives with matrices.

* Allow word (other than end) to precede 'subroutine' declaration.
@krvajal krvajal closed this as completed Jun 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants