sodaware / pangolin.mod Goto Github PK

Just discovered your code to familiarize myself with non-trivial bmax patterns 👍

I just discovered your EventHandlerBag#_grow method:

One thing I implemented last week was a mutable array class that auto-grows when you append elements. bmax-ng's generics fail my all the time, so the resulting code works with plain Objects only, too. I stumbled across ArrayCopy and replaced my for-eachin loops with that.

I don't know how to whip up performance tests, yet, so this might not improve anything. But if it does, replace

		For Local pos:Int = 0 To oldObjects.Length - 1
			Self._objects[pos] = oldObjects[pos]
		Next

with:

		ArrayCopy(oldObjects, 0, Self._objects, 0, oldObjects.Length)

(Note you don't have to -1 here because it's not the index, but the length itself we need.)

Hi Phil,

I really like your coding style, it is very clean and easy to read, I only just happened upon your blitzmax modules,
when I also noticed that the missing pangolin.gfx could contain the type for AbstractRenderRequest.

Thanks in advance