MengbinZhu / stardust-particle-engine

Automatically exported from code.google.com/p/stardust-particle-engine
0 stars 0 forks source link

PooledDisplayObjectClass initializer not working #7

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Using the PooledDisplayObjectClass initializer

What is the expected output? What do you see instead?
Small squares appearing and fading away. Nothing, a null object reference error 
is thrown

What version of the product are you using? On what operating system?
Stardust 1.2.163 on Windows XP SP3

Please provide any additional information below.
Looking at the PooledDisplayObjectClass, all that needed be changed was the 
order of properties being defined. The constructorParams property needed to be 
defined before the displayObjectClass property.

Original issue reported on code.google.com by Kronoshi...@gmail.com on 16 Aug 2010 at 7:07

GoogleCodeExporter commented 9 years ago
I actually fixed this small problem on my own. Due to the nature of the the 
displayObjectClass setter method, Stardust was sending the constructorParams to 
the object pool before defining it, resulting in a null reference error. Simply 
defining constructorParams before the displayObjectClass fixes the problem.

Original comment by Kronoshi...@gmail.com on 16 Aug 2010 at 7:45

GoogleCodeExporter commented 9 years ago
Ah, thanks for the info.
Can you post your fixed version and point out where you've modified the code as 
a reference?

I'm currently on my military service and only have a limited amount of free 
time per week. A reference file would help me speed up the update process.

Thanks.

Original comment by cjcat2266 on 22 Aug 2010 at 3:25

GoogleCodeExporter commented 9 years ago
Alright, here's my fixed version. I have comments in the code to help clear 
things up.

Original comment by Kronoshi...@gmail.com on 22 Aug 2010 at 5:43

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by cjcat2266 on 15 Jan 2011 at 6:15