Opened 8 years ago
Closed 8 years ago
#13239 closed enhancement (fixed)
[Patch] Java 8: MapCSS Condition class
Reported by: | michael2402 | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 16.07 |
Component: | Core mappaint | Version: | |
Keywords: | gsoc-core java8 | Cc: | Don-vip, bastiK, stoecker |
Description
The switch/case in conditions can be removed by using lambda expressions
The speed is not affected by this (the JIT can produce similar assembly code in both cases)
I noticed a small decrease in floating point performance, I think this happens because of the nested lambdas. It is less than 5%, so probably OK.
Performance of EQ seems to have improved extremely (~4 times compared to NEQ). This is because of the JIT thinking that that call is special and optimizing/inlining it. If you change the order of the Ops that are used before the JIT jumps in, you get an other fast entry ;-).
Attachments (1)
Change History (3)
by , 8 years ago
Attachment: | patch-mapcss-java8-condition.patch added |
---|
comment:1 by , 8 years ago
Milestone: | 16.08 → 16.07 |
---|
comment:2 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
In 10674/josm: