That's modifying A as much as it is modifying B though ;-)

Btw, there's already a global function called "GrabContents".

>I don't get your last point. What documentation?

In the code there is a big comment with use cases. I am referring to use case number 2.
Clonk with some arrows and a bow with arrows collects arrow pack. The pack refills the arrows in the bow and infectious inventory, then is collected if there still is space in the inventory.

It does not happen this way. The pack refills the arrows in the bow, nothing else, and does not enter the inventory.

[Update]
I used the original implementation in my unit tests, to serve as a reference. It turns out that the stacks filled objects recursively in the original, just as described in that comment. I had botched something along the way.

I'd like to change the return value of SplitRGBaValue to either

a) return a proplist: {R, G, B, Alpha}, so that the format is compatible with particles and can be added via AddProperties(); or
b) return an array, as it is, but switch the order of the entries to [Alpha, Red, Green, Blue] instead of the current implementation [Red, Green, Blue, Alpha]. With the current implementation you cannot use the constants RGB_RED, etc. to get values from the array.

My personal favorite is option a), because option b) would do the same as GetRGBaValue() then, just in a different way (you get all values at once, but for access you use the same parameters as the function call). The function is never used in the game, but it is an incompatible change, so before I do anything it would be nice to hear your opinions.