Mercurial > mplayer.hg

changeset 1126:4f610bd631ac

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
fixed some possible memleak places
author al3x
date Thu, 14 Jun 2001 12:42:56 +0000
parents 3ec8f4779e81
children ca1bc9edf735
files libvo/vo_sdl.c
diffstat 1 files changed, 33 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/libvo/vo_sdl.c	Wed Jun 13 21:57:36 2001 +0000
+++ b/libvo/vo_sdl.c	Thu Jun 14 12:42:56 2001 +0000
@@ -459,7 +459,7 @@
 
 	/* Free RGB Surface */
 	if (priv->rgbsurface)
-		SDL_FreeSurface(priv->rgbsurface);	
+		SDL_FreeSurface(priv->rgbsurface);
 
 	/* Free our blitting surface */
 	if (priv->surface)
@@ -543,10 +543,11 @@
 	
 	/* if we were successfull hide the mouse cursor and save the mode */
 	if (newsurface) {
+		if (priv->surface)
+	    	    SDL_FreeSurface(priv->surface);
 		priv->surface = newsurface;
 		SDL_ShowCursor(0);
 	}
-	//TODO: check if this produces memhole! (no surface freeing)
 }
 #endif
 
@@ -575,11 +576,13 @@
 	
 	/* if creation of new surface was successfull, save it and hide mouse cursor */
 	if(newsurface) {
+		if (priv->surface)
+	    	    SDL_FreeSurface(priv->surface);
 		priv->surface = newsurface;
 		SDL_ShowCursor(0);
 	}		
 }
-	
+
 
 /**
  * Initialize an SDL surface and an SDL YUV overlay.
@@ -674,7 +677,7 @@
 	    return -1;
 
 	/* Set output window title */
-	SDL_WM_SetCaption (".: MPlayer : F = Fullscreen/Windowed : C = Cycle Fullscreen Resolutions :.", "SDL Video Out");
+	SDL_WM_SetCaption (".: MPlayer : F = Fullscreen/Windowed : C = Cycle Fullscreen Resolutions :.", "MPlayer's SDL Video Out");
 	//SDL_WM_SetCaption (title, title);
 
 	/* Save the original Image size */
@@ -710,6 +713,8 @@
 	if(flags&FS) {
 	  	if(verbose) printf("SDL: setting zoomed fullscreen without modeswitching\n");
 		printf("SDL: Info - please use -vm (unscaled) or -zoom (scaled) for best fullscreen experience\n");
+		if (priv->surface)
+		    SDL_FreeSurface(priv->surface);
           	priv->surface = NULL;
 		priv->fulltype = FS;
 		set_fullmode(priv->fullmode);
@@ -719,7 +724,8 @@
 	if(flags&VM) {
 	 	if(verbose) printf("SDL: setting nonzoomed fullscreen with modeswitching\n");
 		printf("SDL: Info - please use -zoom switch to scale video\n");
-		
+		if (priv->surface)
+		    SDL_FreeSurface(priv->surface);
           	priv->surface = NULL;
 		priv->fulltype = VM;
 		set_fullmode(priv->fullmode);
@@ -729,7 +735,8 @@
 	if(flags&ZOOM) {
 	 	if(verbose) printf("SDL: setting zoomed fullscreen with modeswitching\n");
 		printf("SDL: Info - please use -vm switch instead if you don't want scaled video\n");
-		
+		if (priv->surface)
+		    SDL_FreeSurface(priv->surface);
           	priv->surface = NULL;
 		priv->fulltype = ZOOM;
 		set_fullmode(priv->fullmode);
@@ -746,7 +753,8 @@
 		else {
 			if(verbose) printf("SDL: setting zoomed fullscreen with modeswitching\n");
 			printf("SDL: Info - please use -vm switch instead if you don't want scaled video\n");
-			
+			if (priv->surface)
+			    SDL_FreeSurface(priv->surface);
 			priv->surface = NULL;
 			priv->fulltype = ZOOM;
 			set_fullmode(priv->fullmode);
@@ -768,48 +776,64 @@
 		// 15 bit: r:111110000000000b g:000001111100000b b:000000000011111b
 		// FIXME: colorkey detect based on bpp, FIXME static bpp value, FIXME alpha value correct?
 	    case IMGFMT_RGB15:
+		if (priv->rgbsurface)
+			SDL_FreeSurface(priv->rgbsurface);
 		if (!(priv->rgbsurface = SDL_CreateRGBSurface (SDL_SRCCOLORKEY, width, height, 15, 31, 992, 31744, 0))) {
 			printf ("SDL: Couldn't create a RGB surface: %s\n", SDL_GetError());
 			return -1;
 		}
 	    break;	
 	    case IMGFMT_BGR15:
+		if (priv->rgbsurface)
+			SDL_FreeSurface(priv->rgbsurface);
 		if (!(priv->rgbsurface = SDL_CreateRGBSurface (SDL_SRCCOLORKEY, width, height, 15, 31744, 992, 31, 0))) {
 			printf ("SDL: Couldn't create a RGB surface: %s\n", SDL_GetError());
 			return -1;
 		}
 	    break;	
 	    case IMGFMT_RGB16:
+		if (priv->rgbsurface)
+			SDL_FreeSurface(priv->rgbsurface);
 		if (!(priv->rgbsurface = SDL_CreateRGBSurface (SDL_SRCCOLORKEY, width, height, 16, 31, 2016, 63488, 0))) {
 			printf ("SDL: Couldn't create a RGB surface: %s\n", SDL_GetError());
 			return -1;
 		}
 	    break;	
 	    case IMGFMT_BGR16:
+		if (priv->rgbsurface)
+			SDL_FreeSurface(priv->rgbsurface);
 		if (!(priv->rgbsurface = SDL_CreateRGBSurface (SDL_SRCCOLORKEY, width, height, 16, 63488, 2016, 31, 0))) {
 			printf ("SDL: Couldn't create a RGB surface: %s\n", SDL_GetError());
 			return -1;
 		}
 	    break;	
 	    case IMGFMT_RGB24:
+		if (priv->rgbsurface)
+			SDL_FreeSurface(priv->rgbsurface);
 		if (!(priv->rgbsurface = SDL_CreateRGBSurface (SDL_SRCCOLORKEY, width, height, 24, 0x0000FF, 0x00FF00, 0xFF0000, 0))) {
 			printf ("SDL: Couldn't create a RGB surface: %s\n", SDL_GetError());
 			return -1;
 		}
 	    break;	
 	    case IMGFMT_BGR24:
+		if (priv->rgbsurface)
+			SDL_FreeSurface(priv->rgbsurface);
 		if (!(priv->rgbsurface = SDL_CreateRGBSurface (SDL_SRCCOLORKEY, width, height, 24, 0xFF0000, 0x00FF00, 0x0000FF, 0))) {
 			printf ("SDL: Couldn't create a RGB surface: %s\n", SDL_GetError());
 			return -1;
 		}
 	    break;	
 	    case IMGFMT_RGB32:
+		if (priv->rgbsurface)
+			SDL_FreeSurface(priv->rgbsurface);
 		if (!(priv->rgbsurface = SDL_CreateRGBSurface (SDL_SRCCOLORKEY, width, height, 32, 0x000000FF, 0x0000FF00, 0x00FF0000, 0xFF000000))) {
 			printf ("SDL: Couldn't create a RGB surface: %s\n", SDL_GetError());
 			return -1;
 		}
 	    break;	
 	    case IMGFMT_BGR32:
+		if (priv->rgbsurface)
+			SDL_FreeSurface(priv->rgbsurface);
 		if (!(priv->rgbsurface = SDL_CreateRGBSurface (SDL_SRCCOLORKEY, width, height, 32, 0x00FF0000, 0x0000FF00, 0x000000FF, 0xFF000000))) {
 			printf ("SDL: Couldn't create a RGB surface: %s\n", SDL_GetError());
 			return -1;
@@ -817,6 +841,8 @@
 	    break;	
 	    default:
 		/* Initialize and create the YUV Overlay used for video out */
+		if (priv->overlay)
+			SDL_FreeYUVOverlay(priv->overlay);
 		if (!(priv->overlay = SDL_CreateYUVOverlay (width, height, sdl_format, priv->surface))) {
 			printf ("SDL: Couldn't create a YUV overlay: %s\n", SDL_GetError());
 			return -1;