Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [wtp-dev] About parsing and conversion times



On 7 Apr 2016, at 11:55, Angelo zerr wrote:

Great job Eugene! I hope Google Closure will able to be used inside JSDT
before Neon release.

I would not hold my breath on that. Even if we can get the IP work done. We still need a converter from Closure’s IR to DOM AST so that the current functionality
continues to work.

Good luck Gorkem and many thanks again for your work!

2016-04-07 17:52 GMT+02:00 Gorkem Ercan <gorkem.ercan@xxxxxxxxx>:


This is really good.
I have opened [1] as my first act as JSDT committer :( to get legal
blessing for using it.
—
Gorkem

[1] https://dev.eclipse.org/ipzilla/show_bug.cgi?id=11185



On 7 Apr 2016, at 11:36, Eugene Melekhov wrote:

Hi all,

I've posted some timings for esprima parsing tsserver.js and subsequent
conversion resulting AST into JSDT ASTDom format
here https://bugs.eclipse.org/bugs/show_bug.cgi?id=490898

Angelo asked about testing parsing tsserver.js using Closure compiler, so
I think it's worth presenting all
results for esprima and closure together. All results are in seconds.

Esprima on nashorn + JSDT Dom Conversion
=========================================
Parse   convert
10.738  7.854
18.736  8.316
12.929  5.261
-------
6.194  4.826
6.036  3.830
-------
4.562  4.982
4.675  3.942


Google closure parser + converting into Google's Internal Representation
=========================================
Parse  convert Total
0.620  0.260   0.880


At first I didn't belive it, so I used Google's Code Printer to print
resulting
Internal Representation (after conversion from AST) to make sure that
file was really parsed.
And it really was....

So, result speaks for himself.... :-)

That's why I think we should switch to closure and use its internal
format.

Thanks.

--
Eugene Melekhov

_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe
from this list, visit
https://dev.eclipse.org/mailman/listinfo/wtp-dev

_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe
from this list, visit
https://dev.eclipse.org/mailman/listinfo/wtp-dev

_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/wtp-dev


Back to the top