Fivetonsofflax / opentyrian

Automatically exported from code.google.com/p/opentyrian
GNU General Public License v2.0
0 stars 0 forks source link

void fade_song() not implemented #129

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Finish a level.

What is the expected output? What do you see instead?
I don't expect "TODO: fade_song" to be printed on stdout.

What version of the product are you using? On what operating system?
hg tip on Fedora 17.

Please provide any additional information below.
Obviously, this function is simply not implemented yet. I think it's a good 
idea to either implement it or make it a complete NOP. The "TODO: fade_song" 
print to stdout is somewhat annoying.

Original issue reported on code.google.com by a.h.vand...@gmail.com on 3 Jun 2013 at 7:22

GoogleCodeExporter commented 9 years ago
This also happens when clicking or hitting space on the music entry in the 
pause menu. It doesn't affect anything.

Original comment by parker.l...@gmail.com on 3 Jun 2013 at 7:31

GoogleCodeExporter commented 9 years ago
Yes, I'm aware that it doesn't affect anything per se, but I find the repeated 
"TODO" prints on stdout annoying; a simple "/* TODO */" marker in the code is 
sufficient, spamming stdout isn't needed.

Original comment by a.h.vand...@gmail.com on 3 Jun 2013 at 8:22

GoogleCodeExporter commented 9 years ago
silenced as of rc21a07d4803f

Original comment by mindless...@gmail.com on 6 Aug 2013 at 4:09