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.

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

Ecma script 6 in sidebar class view probably broken #1891

Closed
zero0bytes opened this issue Jul 4, 2018 · 4 comments

Comments

Projects
None yet
3 participants
@zero0bytes
Copy link

commented Jul 4, 2018

Hi !
When i edit ecma script 6 javascript files i see there is maybe broken class view, there is problem when editing big files, i use geany because of its class view lovely feature. Im using geany version 1.33 "Gorgon"

geany
as you can see there is no show class but source contain classes

update:
Here is example javascript ES6 code: https://github.com/zero0bytes/pixi.js-GUI/blob/master/pixi-gui.js

best regards

@b4n

This comment has been minimized.

Copy link
Member

commented Jul 4, 2018

JavaScript is notably hard to analyze statically due to it's inherent dynamic nature, which means that short of actually executing the JavaScript code (and every single possible code path) and discovering new functions such, it's unlikely it could be possible to get fully accurate code structure.

However, in most common cases it's possible to infer what the developer meant, which gets mostly good results. In your case, could you provide a sample JavaScript source so we could try and debug this?
BTW, @universal-ctags/ctags is now the canonical upstream for the code that parses sources, so you might wanna check how it behaves, and report a bug there (I'm also involved there).

@b4n b4n self-assigned this Jul 4, 2018

@zero0bytes

This comment has been minimized.

Copy link
Author

commented Jul 10, 2018

Hi !
Here is example javascript ES6 code: https://github.com/zero0bytes/pixi.js-GUI/blob/master/pixi-gui.js

@elextr

This comment has been minimized.

Copy link
Member

commented Jul 10, 2018

Be nice if you could provide a minimum example, with what you got and what you expect.

@b4n

This comment has been minimized.

Copy link
Member

commented Apr 28, 2019

The missing classes are fixed by ctags sync from #2018.

@b4n b4n closed this Apr 28, 2019

@b4n b4n added this to the 1.35 milestone Apr 28, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.