Closed soywiz closed 7 years ago
Wouldn't it be by far easier to implicitly mark all methods as @JTranscSync
and only async methods as JTranscAsync
?
@intrigus Yes, it would be easier. But I wanted two things:
Merging #274 into master will increase coverage by
0.02%
. The diff coverage is42.99%
.
@@ Coverage Diff @@
## master #274 +/- ##
============================================
+ Coverage 17.82% 17.85% +0.02%
Complexity 386 386
============================================
Files 312 314 +2
Lines 20747 20802 +55
Branches 4485 4493 +8
============================================
+ Hits 3699 3714 +15
- Misses 16369 16405 +36
- Partials 679 683 +4
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
...sc-rt-core/src/com/jtransc/text/MStringReader.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
...re/src/com/jtransc/simd/MutableFloat32x4Utils.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
...c-rt-core/src/com/jtransc/lang/JTranscObjects.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
...ransc-rt-core/src/com/jtransc/mem/FastMemByte.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
jtransc-rt-core/src/com/jtransc/FastMemory.java | 19.35% <ø> (ø) |
6 <0> (ø) |
:arrow_down: |
jtransc-rt-core/src/com/jtransc/simd/Simd.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
...c-rt-core/src/com/jtransc/simd/Float32x4Utils.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
...sc-rt-core/src/com/jtransc/mem/BytesReadAlign.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
jtransc-rt-core/src/com/jtransc/mem/BytesRead.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
...ansc-rt-core/src/com/jtransc/ds/FastStringMap.java | 0% <ø> (ø) |
0 <0> (ø) |
:arrow_down: |
... and 42 more |
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 67193c2...3302723. Read the comment docs.
Target JS to ES6 (you can always transpile it with a transpiler) Use async/await pattern to enable support for synchronous I/O and threads
Partially implementation of https://github.com/jtransc/jtransc/issues/265 (but just for JS)