Evgeniy Rojkov Suggestions. Devs please read.
Posts 37
Added by Evgeniy Rojkov 12 months ago

#1 Parse and regional settings

for god sake, use Parse(X, CultureInfo.InvariantCulture)
there alot cultures (russian and brazil) where used comma insted of dot as number separator.
cocos2d xna use a lot float.Parse and int.Parse for parsing *.plist (sprite sheets animations and particles).
all *.plist use dot as number seprator.
this lead to fatal exceptions on windows phones with russian (and brazil) regional settings.
everyone who want to use cocos-xna with russian (or brazil) marketplaces should fix this by hands, or replace all dots with commas in plist files.

#2 CCLog is a bad practice

I maybe wrong. But i think compiler ommit System.Diagnostics.Debug for release builds.
But you gus replaced all Debug.Trace with CCLog. And all debug information (messages and other info) exist in release builds.
Its not shown anywhere, its not big cost of size or performance, but its still exist in release builds.

p.s. sorry for my bad english.

Jacob Anderson RE: Suggestions. Devs please read.
Posts 178
Location Totally Evil Entertainment - San Diego
Added by Jacob Anderson 10 months ago

https://github.com/cocos2d/cocos2d-x-for-xna/issues/18

Issues 17 and 18 in the github tracker.

Jacob Anderson RE: Suggestions. Devs please read.
Posts 178
Location Totally Evil Entertainment - San Diego
Added by Jacob Anderson 10 months ago

Item #2 - CCLog - this still uses Debug.WriteLine(string), so the output should not appear in your application when running in Release mode.

Jacob Anderson RE: Suggestions. Devs please read.
Posts 178
Location Totally Evil Entertainment - San Diego
Added by Jacob Anderson 10 months ago

Item #1 for both int and float is in version 0.1.2a.


(1-3/3)