comparison sv/videoio/SDL_ffmpeg.cpp @ 170:caaddda5477c

- window size limited to a multiple of 4
- allow to have a bigger image than original,
- add several cleans on pointers

     // create a packet for our data
     AVPacket pack;
 
     // reserve some pointers for use in loop
     AVFrame *inFrame, *inFrameRGB;
+	uint8_t *inVideoBuffer = NULL;
 
     // allocate a frame
     inFrame = avcodec_alloc_frame();
 
     // allocate another frame for unknown->RGB conversion
     inFrameRGB = avcodec_alloc_frame();
 
     if(SDL_ffmpegValidVideo(file)) {
         // allocate buffer
-        uint8_t *inVideoBuffer = (uint8_t*)malloc(  avpicture_get_size(file->vs[file->videoStream]->pixFmt,
+        inVideoBuffer = (uint8_t*)malloc(  avpicture_get_size(file->vs[file->videoStream]->pixFmt,
                                                         file->vs[file->videoStream]->width,
                                                         file->vs[file->videoStream]->height) );
 
         // put buffer into our reserved frame
         avpicture_fill( (AVPicture*)inFrameRGB,

                 // if new timestamp is from future, we proceed
 			//	if(buf->timestamp >= SDL_ffmpegGetPosition(file)) 
 			//	{
 				int w=(int)(zoomFivan*320+0.5);
 				int h=(int)(zoomFivan*240+0.5);
-				if ((w>file->vs[file->videoStream]->width)||(h>file->vs[file->videoStream]->height)){
-					w=file->vs[file->videoStream]->width;
-					h=file->vs[file->videoStream]->height;
-				}
+				//if ((w>file->vs[file->videoStream]->width)||(h>file->vs[file->videoStream]->height)){
+				//	w=file->vs[file->videoStream]->width;
+				//	h=file->vs[file->videoStream]->height;
+				//}
 				// Be sure we have a multiple of 4
 				w &= 0xFFFFFFFC;
 				h &= 0xFFFFFFFC;
 				if (img_convert_ctx == NULL) {
 

 							if (img_convert_ctx == NULL) {
 								fprintf(stderr, "Cannot initialize the conversion context\n");
 								exit(1);
 							}
 					}
-				
+								
+					// check to see if buffer is at the same size than the screen
+					if (inFrameRGB->linesize[0]/3 != w ) {
+						av_free(inFrameRGB);
+						free(inVideoBuffer);
+						//avcodec_default_release_buffer(img_convert_ctx , inFrameRGB);
+						inFrameRGB = avcodec_alloc_frame();
+						// allocate buffer
+						inVideoBuffer = (uint8_t*)malloc(  avpicture_get_size(file->vs[file->videoStream]->pixFmt,
+                                                        w,
+                                                        h) );
+
+						// put buffer into our reserved frame
+						avpicture_fill( (AVPicture*)inFrameRGB,
+										inVideoBuffer,
+										file->vs[file->videoStream]->pixFmt,
+										w,
+										h);
+
+					}
+					
 					((AVPicture*)inFrameRGB)->linesize[0]=(int)w*3;
-				sws_scale(img_convert_ctx, ((AVPicture*)inFrame)->data, ((AVPicture*)inFrame)->linesize,
-                      0, file->vs[file->videoStream]->height, ((AVPicture*)inFrameRGB)->data, ((AVPicture*)inFrameRGB)->linesize);
-				 sws_freeContext(img_convert_ctx);
-				 img_convert_ctx=NULL;
-                    // we convert whatever type of data we got to RGB24
+					sws_scale(img_convert_ctx, 
+						((AVPicture*)inFrame)->data, ((AVPicture*)inFrame)->linesize, 0, file->vs[file->videoStream]->height, 
+						((AVPicture*)inFrameRGB)->data, ((AVPicture*)inFrameRGB)->linesize);
+				 
+					sws_freeContext(img_convert_ctx);
+					img_convert_ctx=NULL;
+
+					// we convert whatever type of data we got to RGB24
                   /* img_convert((AVPicture*)inFrameRGB,
                         file->vs[file->videoStream]->pixFmt,
                         (AVPicture*)inFrame,
                         ((AVCodecContext*)file->vs[file->videoStream]->_ffmpeg)->pix_fmt,
                         file->vs[file->videoStream]->width,
                         file->vs[file->videoStream]->height);
 */
+
                     // allocate image room
                     buf->img = SDL_CreateRGBSurface(SDL_SWSURFACE,
                         w,
                         h,
                         24, 0x0000FF, 0x00FF00, 0xFF0000, 0);
-
                     // copy image data to image room
                     memcpy(buf->img->pixels, inFrameRGB->data[0],
                         w*h* 3);
 					file->timebase=buf->timestamp-file->vs[file->videoStream]->lastTimeStamp;
                     // we write the lastTimestamp we got

         // we leave the decode semaphore
         SDL_SemPost(file->decode);
 		if ((file->skipAudio)&&(file->delay)) 
 			SDL_Delay(3);
     }
+
+	if (inVideoBuffer)
+	{
+		free(inVideoBuffer);
+		inVideoBuffer = NULL;
+	}
+
     // if we stop this thread, we can release the packet we reserved
     av_free_packet(&pack);
+	free(samples);
+	av_free(inFrameRGB);
+	av_free(inFrame);
 
     return 0;
 }
 
 int SDL_ffmpegSeek(SDL_ffmpegFile* file, int64_t timestamp) {