Mahtan/wagic
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fixed a memleak in AdWhirlConfig.m

Browse Source 
added a TODO about a compiler warning I received that was not flagged in the code.
This commit is contained in:
techdragon.nguyen@gmail.com
2012-01-10 20:21:57 +00:00
parent adc3e4b689
commit bb2536512a
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
adwhirl/AdWhirlSDK_iOS_3.1.1/AdWhirl/internal/AdWhirlConfig.m
View File

@@ -327,7 +327,8 @@ BOOL awDoubleVal(double *var, id val) {
case 3: case 3:
// general config map // general config map
if (![self parseExtraConfig:configDict error:error]) { if (![self parseExtraConfig:configDict error:error]) {
return NO; [adNetConfigDicts release];
return NO;
} }
break; // general config map break; // general config map
default: default:

3
projects/mtg/src/GameOptions.cpp
View File

@@ -251,7 +251,8 @@ PIXEL_TYPE GameOption::asColor(PIXEL_TYPE fallback)
color[subpixel] = (unsigned char) atoi(temp.c_str()); color[subpixel] = (unsigned char) atoi(temp.c_str());
if (subpixel == 2) if (subpixel == 2)
color[3] = 255; color[3] = 255;
// TODO: WARNING - not all the values of color may be initialized at this point. Do we need to initialize the array to some value?
return ARGB(color[3],color[0],color[1],color[2]); return ARGB(color[3],color[0],color[1],color[2]);
} }