diff options
| author | Sam Lantinga <slouken@libsdl.org> | 2012-01-10 22:52:17 -0500 |
|---|---|---|
| committer | Sam Lantinga <slouken@libsdl.org> | 2012-01-10 22:52:17 -0500 |
| commit | 334f648b1479c4cab6c887604e3f67930966311d (patch) | |
| tree | 6a94df6926c902708e1ec99100fdd758948b9253 | |
| parent | 77fbbe5c16c2eb911ae1e7d41fcd026fd00bd1b5 (diff) | |
| download | sdl-334f648b1479c4cab6c887604e3f67930966311d.tar.gz | |
Fixed typo in mask comparison
Bl0ckeduser
I downloaded the SDL 1.2.15 prerelease and compiled it (under Mac OS
Classic). All the tests worked without issues, but the compiler gave
this warning [formatting edited for email format]:
} else if ( vf->Rmask == 0xFF00 && (vf->Bmask = 0xFF000000) ) {
":src:video:SDL_video.c"; line 990 #Warning 2: possible unintended assignment
I discussed this with another programmer and we came to the conclusion
that this is probably a typo.
I have checked the latest mercurial sources, and the possible typo is
still there.
Bl0ckeduser
| -rw-r--r-- | src/video/SDL_video.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/video/SDL_video.c b/src/video/SDL_video.c index b226e0376..46285c990 100644 --- a/src/video/SDL_video.c +++ b/src/video/SDL_video.c @@ -987,7 +987,7 @@ SDL_Surface *SDL_DisplayFormatAlpha(SDL_Surface *surface) if ( (vf->Rmask == 0xff) && (vf->Bmask == 0xff0000) ) { rmask = 0xff; bmask = 0xff0000; - } else if ( vf->Rmask == 0xFF00 && (vf->Bmask = 0xFF000000) ) { + } else if ( vf->Rmask == 0xFF00 && (vf->Bmask == 0xFF000000) ) { amask = 0x000000FF; rmask = 0x0000FF00; gmask = 0x00FF0000; |