search

297chrisc / playn

Automatically exported from code.google.com/p/playn
0 stars 0 forks source link

AbstractSound never sets soundLoaded #155

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Load sound once
2. Load it again
3. Notice that the onComplete callback is not invoked

What is the expected output? What do you see instead?
Loading an already loaded Sound should trigger the ResourceCallback again

What version of the product are you using? On what operating system?
trunk

Please provide any additional information below.

I guess setting soundLoaded inside AbstracSound#onLoadComplete solves it.

Original issue reported on code.google.com by christop...@gmail.com on 27 Mar 2012 at 2:27

GoogleCodeExporter commented 9 years ago 
Oh, this is a duplicate of #143.

Original comment by christop...@gmail.com on 27 Mar 2012 at 6:11

GoogleCodeExporter commented 9 years ago 
Yes, I also noticed this bug.
It only happens on Android/HTML5 implementation of PlayN and can be resolved by 
properly setting the "soundLoaded" boolean member of the AbstractSound class, 
as you already noticed. :thumbup:

Maybe I'll provide a patch for this little fix if I can find some time to do 
so, or if you have time to, you can also provide one ! ;-)

Original comment by mbarbe...@gmail.com on 27 Mar 2012 at 7:34

GoogleCodeExporter commented 9 years ago

Original comment by fredsa@google.com on 29 Mar 2012 at 1:24