curioustorvald/Terrarum
1
0
Fork 0
mirror of https://github.com/curioustorvald/Terrarum.git synced 2026-03-16 08:36:07 +09:00
Code Issues Packages Projects Releases Wiki Activity

unsafe array on lightmaprenderer: could this be possible?

Browse Source
This commit is contained in:
minjaesong
2019-06-08 03:33:56 +09:00
parent 9db6e86ac3
commit b16c23f1f0
2 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/net/torvald/terrarum/gameworld/BlockLayer.kt
View File

@@ -5,7 +5,10 @@ import net.torvald.terrarum.AppLoader.printdbg
import sun.misc.Unsafe import sun.misc.Unsafe
/** /**
* Created by minjaesong on 2016-01-17. * Original version Created by minjaesong on 2016-01-17.
* Unsafe version Created by minjaesong on 2019-06-08.
*
* Note to self: refrain from using shorts--just do away with two bytes: different system have different endianness
*/ */
open class BlockLayer(val width: Int, val height: Int) : Disposable { open class BlockLayer(val width: Int, val height: Int) : Disposable {

2
src/net/torvald/terrarum/worlddrawer/LightmapRendererNew.kt
View File

@@ -147,6 +147,7 @@ object LightmapRenderer {
val ypos = y - for_y_start + overscan_open val ypos = y - for_y_start + overscan_open
val xpos = x - for_x_start + overscan_open val xpos = x - for_x_start + overscan_open
// TODO as you can see above, we're already doing a boundary check; try using unsafe here?
return lightmap[ypos][xpos] return lightmap[ypos][xpos]
//return lightmap[ypos * LIGHTMAP_WIDTH + xpos] //return lightmap[ypos * LIGHTMAP_WIDTH + xpos]
} }
@@ -174,6 +175,7 @@ object LightmapRenderer {
val ypos = y - for_y_start + overscan_open val ypos = y - for_y_start + overscan_open
val xpos = x - for_x_start + overscan_open val xpos = x - for_x_start + overscan_open
// TODO as you can see above, we're already doing a boundary check; try using unsafe here?
lightmap[ypos][xpos] = applyFun.invoke(list[ypos][xpos], colour) lightmap[ypos][xpos] = applyFun.invoke(list[ypos][xpos], colour)
//list[ypos * LIGHTMAP_WIDTH + xpos] = applyFun.invoke(list[ypos * LIGHTMAP_WIDTH + xpos], colour) //list[ypos * LIGHTMAP_WIDTH + xpos] = applyFun.invoke(list[ypos * LIGHTMAP_WIDTH + xpos], colour)
} }