soywiz-archive / jtransc

Bytecode to source converting Java & Kotlin code into JavaScript, C++, D, C#, PHP, AS3, Dart and Haxe and run it everywhere. Also use JVM code in your favourite language as a library.
https://jtransc.soywiz.com/
Apache License 2.0
632 stars 67 forks source link

Allow to enable/disable JS asynchronous generation #277

Closed soywiz closed 7 years ago

soywiz commented 7 years ago

Since this affects negatively to performance (specially before detecting synchronous paths is implemented), this should be opt-in.

For example. Benchmark:

TOTAL time: 6615.0 <--- without asynchronous
TOTAL time: 30309.0 <--- with asynchronous without optimizations (5x slower)
codecov[bot] commented 7 years ago

Codecov Report

Merging #277 into master will increase coverage by 0.06%. The diff coverage is 84.61%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #277      +/-   ##
============================================
+ Coverage     17.86%   17.93%   +0.06%     
  Complexity      386      386              
============================================
  Files           314      314              
  Lines         20809    20825      +16     
  Branches       4495     4501       +6     
============================================
+ Hits           3718     3734      +16     
- Misses        16407    16408       +1     
+ Partials        684      683       -1
Impacted Files Coverage Δ Complexity Δ
...nsc-rt-core/src/com/jtransc/io/JTranscConsole.java 0% <ø> (ø) 0 <0> (ø) :arrow_down:
...transc-rt-core/src/com/jtransc/JTranscWrapped.java 0% <ø> (ø) 0 <0> (ø) :arrow_down:
...-core/src/com/jtransc/text/JTranscStringTools.java 0% <ø> (ø) 0 <0> (ø) :arrow_down:
...core/src/com/jtransc/gen/common/CommonGenerator.kt 0% <0%> (ø) 0 <0> (ø) :arrow_down:
jtransc-gen-js/src/com/jtransc/gen/js/JsTarget.kt 86.81% <89.18%> (+1%) 4 <0> (ø) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3ae587f...58b711c. Read the comment docs.