[Ffmpeg-cvslog] r7306 - trunk/libavformat/x11grab.c

gpoirier subversion
Wed Dec 13 09:48:57 CET 2006


Author: gpoirier
Date: Wed Dec 13 09:48:56 2006
New Revision: 7306

Modified:
   trunk/libavformat/x11grab.c

Log:
RFC 3&4 stage: FFmpeg style aint't my style

- Tabs are 4 width
- No tabs allowed
(patch by Edouard Gomez)


Modified: trunk/libavformat/x11grab.c
==============================================================================
--- trunk/libavformat/x11grab.c	(original)
+++ trunk/libavformat/x11grab.c	Wed Dec 13 09:48:56 2006
@@ -8,7 +8,7 @@
  *                    Edouard Gomez <ed.gomez at free.fr>
  *
  * This file contains code from grab.c:
- * Copyright (c) 2000-2001 Fabrice Bellard 
+ * Copyright (c) 2000-2001 Fabrice Bellard
  *
  * This file contains code from the xvidcap project:
  * Copyright (C) 1997-1998 Rasca, Berlin
@@ -48,273 +48,273 @@
 
 typedef struct
 {
-	Display *dpy;
-	int frame_format;
-	int frame_size;
-	int frame_rate;
-	int frame_rate_base;
-	int64_t time_frame;
-
-	int height;
-	int width;
-	int x_off;
-	int y_off;
-	XImage *image;
-	int use_shm;
-	XShmSegmentInfo shminfo;
-	int mouse_wanted;
+    Display *dpy;
+    int frame_format;
+    int frame_size;
+    int frame_rate;
+    int frame_rate_base;
+    int64_t time_frame;
+
+    int height;
+    int width;
+    int x_off;
+    int y_off;
+    XImage *image;
+    int use_shm;
+    XShmSegmentInfo shminfo;
+    int mouse_wanted;
 } X11Grab;
 
 static int
 x11grab_read_header(AVFormatContext *s1, AVFormatParameters *ap)
 {
-	X11Grab *x11grab = s1->priv_data;
-	Display *dpy;
-	AVStream *st = NULL;
-	int width, height;
-	int frame_rate, frame_rate_base, frame_size;
-	int input_pixfmt;
-	XImage *image;
-	int x_off=0; int y_off = 0;
-	int use_shm;
-
-	dpy = XOpenDisplay(NULL);
-	if(!dpy) {
-		goto fail;
-	}
-
-	sscanf(ap->device, "x11:%d,%d", &x_off, &y_off);
-	av_log(s1, AV_LOG_INFO, "device: %s -> x: %d y: %d width: %d height: %d\n", ap->device, x_off, y_off, ap->width, ap->height);
-  
-	if (!ap || ap->width <= 0 || ap->height <= 0 || ap->time_base.den <= 0) {
-		av_log(s1, AV_LOG_ERROR, "AVParameters don't have any video size. Use -s.\n");
-		return AVERROR_IO;  
-	}
-
-	width = ap->width;
-	height = ap->height;
-	frame_rate = ap->time_base.den;
-	frame_rate_base = ap->time_base.num;
-
-	st = av_new_stream(s1, 0);
-	if (!st) {
-		return -ENOMEM;
-	}
-	av_set_pts_info(st, 64, 1, 1000000); /* 64 bits pts in us */
-
-	use_shm = XShmQueryExtension(dpy);
-	av_log(s1, AV_LOG_INFO, "shared memory extension %s\n", use_shm ? "found" : "not found");
-
-	if(use_shm) {
-		int scr = XDefaultScreen(dpy);
-		image = XShmCreateImage(dpy,
-					DefaultVisual(dpy, scr),
-					DefaultDepth(dpy, scr),
-					ZPixmap,
-					NULL,
-					&x11grab->shminfo,
-					ap->width, ap->height);
-		x11grab->shminfo.shmid = shmget(IPC_PRIVATE,
-						image->bytes_per_line * image->height,
-						IPC_CREAT|0777);
-		if (x11grab->shminfo.shmid == -1) {
-			av_log(s1, AV_LOG_ERROR, "Fatal: Can't get shared memory!\n");
-			return -ENOMEM;
-		}
-		x11grab->shminfo.shmaddr = image->data = shmat(x11grab->shminfo.shmid, 0, 0);
-		x11grab->shminfo.readOnly = False;
-            
-		if (!XShmAttach(dpy, &x11grab->shminfo)) {
-			av_log(s1, AV_LOG_ERROR, "Fatal: Failed to attach shared memory!\n");
-			/* needs some better error subroutine :) */
-			return AVERROR_IO;  
-		}
-	} else {
-		image = XGetImage(dpy, RootWindow(dpy, DefaultScreen(dpy)),
-				  x_off,y_off,
-				  ap->width,ap->height,
-				  AllPlanes, ZPixmap); 
-	}
-  
-	switch (image->bits_per_pixel) {
-	case 8:
-		av_log (s1, AV_LOG_DEBUG, "8 bit pallete\n");
-		input_pixfmt = PIX_FMT_PAL8;
-		break;
-	case 16:
-		if ( image->red_mask == 0xF800 && image->green_mask == 0x07E0
-		     && image->blue_mask == 0x1F ) {
-			av_log (s1, AV_LOG_DEBUG, "16 bit RGB565\n");
-			input_pixfmt = PIX_FMT_RGB565;
-		} else if ( image->red_mask == 0x7C00 && 
-			    image->green_mask == 0x03E0 && 
-			    image->blue_mask == 0x1F ) {
-			av_log(s1, AV_LOG_DEBUG, "16 bit RGB555\n");
-			input_pixfmt = PIX_FMT_RGB555;
-		} else {
-			av_log(s1, AV_LOG_ERROR, "RGB ordering at image depth %i not supported ... aborting\n", image->bits_per_pixel);
-			av_log(s1, AV_LOG_ERROR, "color masks: r 0x%.6lx g 0x%.6lx b 0x%.6lx\n", image->red_mask, image->green_mask, image->blue_mask);
-			return AVERROR_IO;  
-		}
-		break;
-	case 24:
-		if ( image->red_mask == 0xFF0000 && 
-		     image->green_mask == 0xFF00
-		     && image->blue_mask == 0xFF ) {
-			input_pixfmt = PIX_FMT_BGR24;
-		} else if ( image->red_mask == 0xFF && image->green_mask == 0xFF00
-			    && image->blue_mask == 0xFF0000 ) {
-			input_pixfmt = PIX_FMT_RGB24;
-		} else {
-			av_log(s1, AV_LOG_ERROR,"rgb ordering at image depth %i not supported ... aborting\n", image->bits_per_pixel);
-			av_log(s1, AV_LOG_ERROR, "color masks: r 0x%.6lx g 0x%.6lx b 0x%.6lx\n", image->red_mask, image->green_mask, image->blue_mask);
-			return AVERROR_IO;  
-		}
-		break;
-	case 32:
+    X11Grab *x11grab = s1->priv_data;
+    Display *dpy;
+    AVStream *st = NULL;
+    int width, height;
+    int frame_rate, frame_rate_base, frame_size;
+    int input_pixfmt;
+    XImage *image;
+    int x_off=0; int y_off = 0;
+    int use_shm;
+
+    dpy = XOpenDisplay(NULL);
+    if(!dpy) {
+        goto fail;
+    }
+
+    sscanf(ap->device, "x11:%d,%d", &x_off, &y_off);
+    av_log(s1, AV_LOG_INFO, "device: %s -> x: %d y: %d width: %d height: %d\n", ap->device, x_off, y_off, ap->width, ap->height);
+
+    if (!ap || ap->width <= 0 || ap->height <= 0 || ap->time_base.den <= 0) {
+        av_log(s1, AV_LOG_ERROR, "AVParameters don't have any video size. Use -s.\n");
+        return AVERROR_IO;
+    }
+
+    width = ap->width;
+    height = ap->height;
+    frame_rate = ap->time_base.den;
+    frame_rate_base = ap->time_base.num;
+
+    st = av_new_stream(s1, 0);
+    if (!st) {
+        return -ENOMEM;
+    }
+    av_set_pts_info(st, 64, 1, 1000000); /* 64 bits pts in us */
+
+    use_shm = XShmQueryExtension(dpy);
+    av_log(s1, AV_LOG_INFO, "shared memory extension %s\n", use_shm ? "found" : "not found");
+
+    if(use_shm) {
+        int scr = XDefaultScreen(dpy);
+        image = XShmCreateImage(dpy,
+                                DefaultVisual(dpy, scr),
+                                DefaultDepth(dpy, scr),
+                                ZPixmap,
+                                NULL,
+                                &x11grab->shminfo,
+                                ap->width, ap->height);
+        x11grab->shminfo.shmid = shmget(IPC_PRIVATE,
+                                        image->bytes_per_line * image->height,
+                                        IPC_CREAT|0777);
+        if (x11grab->shminfo.shmid == -1) {
+            av_log(s1, AV_LOG_ERROR, "Fatal: Can't get shared memory!\n");
+            return -ENOMEM;
+        }
+        x11grab->shminfo.shmaddr = image->data = shmat(x11grab->shminfo.shmid, 0, 0);
+        x11grab->shminfo.readOnly = False;
+
+        if (!XShmAttach(dpy, &x11grab->shminfo)) {
+            av_log(s1, AV_LOG_ERROR, "Fatal: Failed to attach shared memory!\n");
+            /* needs some better error subroutine :) */
+            return AVERROR_IO;
+        }
+    } else {
+        image = XGetImage(dpy, RootWindow(dpy, DefaultScreen(dpy)),
+                          x_off,y_off,
+                          ap->width,ap->height,
+                          AllPlanes, ZPixmap);
+    }
+
+    switch (image->bits_per_pixel) {
+    case 8:
+        av_log (s1, AV_LOG_DEBUG, "8 bit pallete\n");
+        input_pixfmt = PIX_FMT_PAL8;
+        break;
+    case 16:
+        if ( image->red_mask == 0xF800 && image->green_mask == 0x07E0
+             && image->blue_mask == 0x1F ) {
+            av_log (s1, AV_LOG_DEBUG, "16 bit RGB565\n");
+            input_pixfmt = PIX_FMT_RGB565;
+        } else if ( image->red_mask == 0x7C00 &&
+                    image->green_mask == 0x03E0 &&
+                    image->blue_mask == 0x1F ) {
+            av_log(s1, AV_LOG_DEBUG, "16 bit RGB555\n");
+            input_pixfmt = PIX_FMT_RGB555;
+        } else {
+            av_log(s1, AV_LOG_ERROR, "RGB ordering at image depth %i not supported ... aborting\n", image->bits_per_pixel);
+            av_log(s1, AV_LOG_ERROR, "color masks: r 0x%.6lx g 0x%.6lx b 0x%.6lx\n", image->red_mask, image->green_mask, image->blue_mask);
+            return AVERROR_IO;
+        }
+        break;
+    case 24:
+        if ( image->red_mask == 0xFF0000 &&
+             image->green_mask == 0xFF00
+             && image->blue_mask == 0xFF ) {
+            input_pixfmt = PIX_FMT_BGR24;
+        } else if ( image->red_mask == 0xFF && image->green_mask == 0xFF00
+                    && image->blue_mask == 0xFF0000 ) {
+            input_pixfmt = PIX_FMT_RGB24;
+        } else {
+            av_log(s1, AV_LOG_ERROR,"rgb ordering at image depth %i not supported ... aborting\n", image->bits_per_pixel);
+            av_log(s1, AV_LOG_ERROR, "color masks: r 0x%.6lx g 0x%.6lx b 0x%.6lx\n", image->red_mask, image->green_mask, image->blue_mask);
+            return AVERROR_IO;
+        }
+        break;
+    case 32:
 #if 0
-		GetColorInfo (image, &c_info);
-		if ( c_info.alpha_mask == 0xFF000000 && image->green_mask == 0xFF00 ) {
-			/* byte order is relevant here, not endianness
-			 * endianness is handled by avcodec, but atm no such thing
-			 * as having ABGR, instead of ARGB in a word. Since we
-			 * need this for Solaris/SPARC, but need to do the conversion
-			 * for every frame we do it outside of this loop, cf. below
-			 * this matches both ARGB32 and ABGR32 */
-			input_pixfmt = PIX_FMT_ARGB32;
-		}  else {
-			av_log(s1, AV_LOG_ERROR,"image depth %i not supported ... aborting\n", image->bits_per_pixel);
-			return AVERROR_IO;  
-		}
+        GetColorInfo (image, &c_info);
+        if ( c_info.alpha_mask == 0xFF000000 && image->green_mask == 0xFF00 ) {
+            /* byte order is relevant here, not endianness
+             * endianness is handled by avcodec, but atm no such thing
+             * as having ABGR, instead of ARGB in a word. Since we
+             * need this for Solaris/SPARC, but need to do the conversion
+             * for every frame we do it outside of this loop, cf. below
+             * this matches both ARGB32 and ABGR32 */
+            input_pixfmt = PIX_FMT_ARGB32;
+        }  else {
+            av_log(s1, AV_LOG_ERROR,"image depth %i not supported ... aborting\n", image->bits_per_pixel);
+            return AVERROR_IO;
+        }
 #endif
-		input_pixfmt = PIX_FMT_RGBA32;
-		break;
-	default:
-		av_log(s1, AV_LOG_ERROR, "image depth %i not supported ... aborting\n", image->bits_per_pixel);
-		return -1;
-	}
-
-	frame_size = width * height * image->bits_per_pixel/8;
-	x11grab->frame_size = frame_size;
-	x11grab->dpy = dpy;
-	x11grab->width = ap->width;
-	x11grab->height = ap->height;
-	x11grab->frame_rate      = frame_rate;
-	x11grab->frame_rate_base = frame_rate_base;
-	x11grab->time_frame = av_gettime() * frame_rate / frame_rate_base;
-	x11grab->x_off = x_off;
-	x11grab->y_off = y_off;
-	x11grab->image = image;
-	x11grab->use_shm = use_shm;
-	x11grab->mouse_wanted = 1;
-
-	st->codec->codec_type = CODEC_TYPE_VIDEO;
-	st->codec->codec_id = CODEC_ID_RAWVIDEO;
-	st->codec->width = width;
-	st->codec->height = height;
-	st->codec->pix_fmt = input_pixfmt;
-	st->codec->time_base.den = frame_rate;
-	st->codec->time_base.num = frame_rate_base;
-	st->codec->bit_rate = frame_size * 1/av_q2d(st->codec->time_base) * 8;
-  
-	return 0;
+        input_pixfmt = PIX_FMT_RGBA32;
+        break;
+    default:
+        av_log(s1, AV_LOG_ERROR, "image depth %i not supported ... aborting\n", image->bits_per_pixel);
+        return -1;
+    }
+
+    frame_size = width * height * image->bits_per_pixel/8;
+    x11grab->frame_size = frame_size;
+    x11grab->dpy = dpy;
+    x11grab->width = ap->width;
+    x11grab->height = ap->height;
+    x11grab->frame_rate      = frame_rate;
+    x11grab->frame_rate_base = frame_rate_base;
+    x11grab->time_frame = av_gettime() * frame_rate / frame_rate_base;
+    x11grab->x_off = x_off;
+    x11grab->y_off = y_off;
+    x11grab->image = image;
+    x11grab->use_shm = use_shm;
+    x11grab->mouse_wanted = 1;
+
+    st->codec->codec_type = CODEC_TYPE_VIDEO;
+    st->codec->codec_id = CODEC_ID_RAWVIDEO;
+    st->codec->width = width;
+    st->codec->height = height;
+    st->codec->pix_fmt = input_pixfmt;
+    st->codec->time_base.den = frame_rate;
+    st->codec->time_base.num = frame_rate_base;
+    st->codec->bit_rate = frame_size * 1/av_q2d(st->codec->time_base) * 8;
+
+    return 0;
 fail:
-	av_free(st);
-	return AVERROR_IO;  
+    av_free(st);
+    return AVERROR_IO;
 }
 
 static void
 getCurrentPointer(AVFormatContext *s1, X11Grab *s, int *x, int *y)
 {
-	Window mrootwindow, childwindow;
-	int dummy;
-	Display *dpy = s->dpy;
-  
-	mrootwindow = DefaultRootWindow(dpy);
-  
-	if (XQueryPointer(dpy, mrootwindow, &mrootwindow, &childwindow,
-			  x, y, &dummy, &dummy, (unsigned int*)&dummy)) {
-	} else {
-		av_log(s1, AV_LOG_INFO, "couldn't find mouse pointer\n");
-		*x = -1;
-		*y = -1;
-	}
+    Window mrootwindow, childwindow;
+    int dummy;
+    Display *dpy = s->dpy;
+
+    mrootwindow = DefaultRootWindow(dpy);
+
+    if (XQueryPointer(dpy, mrootwindow, &mrootwindow, &childwindow,
+                      x, y, &dummy, &dummy, (unsigned int*)&dummy)) {
+    } else {
+        av_log(s1, AV_LOG_INFO, "couldn't find mouse pointer\n");
+        *x = -1;
+        *y = -1;
+    }
 }
 
-#define DRAW_CURSOR_TEMPLATE(type_t) \
-do { \
-type_t *cursor; \
-int width_cursor; \
-uint16_t bm_b, bm_w, mask; \
-\
-for (line = 0; line < min(20, (y_off + height) - *y); line++ ) { \
-	if (s->mouse_wanted == 1) { \
-		bm_b = mousePointerBlack[line]; \
-		bm_w = mousePointerWhite[line]; \
-	} else { \
-		bm_b = mousePointerWhite[line]; \
-		bm_w = mousePointerBlack[line]; \
-	} \
-	mask = ( 0x0001 << 15 ); \
-\
-	for (cursor = (type_t*) im_data, width_cursor = 0; \
-	     ((width_cursor + *x) < (width + x_off) && width_cursor < 16); \
-	     cursor++, width_cursor++) { \
-		if ( ( bm_b & mask ) > 0 ) { \
-			*cursor &= ((~ image->red_mask) & (~ image->green_mask) & (~image->blue_mask )); \
-		} else if ( ( bm_w & mask ) > 0 ) { \
-			*cursor |= (image->red_mask | image->green_mask | image->blue_mask ); \
-		} \
-		mask >>= 1; \
-	} \
-	im_data += image->bytes_per_line; \
-} \
-} while (0)
+#define DRAW_CURSOR_TEMPLATE(type_t)									\
+    do {                                                                \
+        type_t *cursor;                                                 \
+        int width_cursor;                                               \
+        uint16_t bm_b, bm_w, mask;                                      \
+																		\
+        for (line = 0; line < min(20, (y_off + height) - *y); line++ ) { \
+            if (s->mouse_wanted == 1) {                                 \
+                bm_b = mousePointerBlack[line];                         \
+                bm_w = mousePointerWhite[line];                         \
+            } else {                                                    \
+                bm_b = mousePointerWhite[line];                         \
+                bm_w = mousePointerBlack[line];                         \
+            }                                                           \
+            mask = ( 0x0001 << 15 );                                    \
+																		\
+            for (cursor = (type_t*) im_data, width_cursor = 0;          \
+                 ((width_cursor + *x) < (width + x_off) && width_cursor < 16); \
+                 cursor++, width_cursor++) {                            \
+                if ( ( bm_b & mask ) > 0 ) {                            \
+                    *cursor &= ((~ image->red_mask) & (~ image->green_mask) & (~image->blue_mask )); \
+                } else if ( ( bm_w & mask ) > 0 ) {                     \
+                    *cursor |= (image->red_mask | image->green_mask | image->blue_mask ); \
+                }                                                       \
+                mask >>= 1;                                             \
+            }                                                           \
+            im_data += image->bytes_per_line;                           \
+        }                                                               \
+    } while (0)
 
 static void
 paintMousePointer(AVFormatContext *s1, X11Grab *s, int *x, int *y, XImage *image)
 {
-	static const uint16_t const mousePointerBlack[] =
-		{
-			0, 49152, 40960, 36864, 34816,
-			33792, 33280, 33024, 32896, 32832,
-			33728, 37376, 43264, 51456, 1152,
-			1152, 576, 576, 448, 0
-		};
-
-	static const uint16_t const mousePointerWhite[] =
-		{
-			0, 0, 16384, 24576, 28672,
-			30720, 31744, 32256, 32512, 32640,
-			31744, 27648, 17920, 1536, 768,
-			768, 384, 384, 0, 0
-		};
-
-	int x_off = s->x_off;
-	int y_off = s->y_off;
-	int width = s->width;
-	int height = s->height;
-
-	if (   (*x - x_off) >= 0 && *x < (width + x_off)
-	    && (*y - y_off) >= 0 && *y < (height + y_off) ) { 
-		int line;
-		uint8_t *im_data = (uint8_t*)image->data;
-    
-		im_data += (image->bytes_per_line * (*y - y_off)); // shift to right line
-		im_data += (image->bits_per_pixel / 8 * (*x - x_off)); // shift to right pixel
-    
-		switch(image->bits_per_pixel) {
-		case 32:
-			DRAW_CURSOR_TEMPLATE(uint32_t);
-			break;
-		case 16:
-			DRAW_CURSOR_TEMPLATE(uint16_t);
-			break;
-		default:
-			/* XXX: How do we deal with 24bit and 8bit modes ? */
-			break;
-		}
-	}
+    static const uint16_t const mousePointerBlack[] =
+        {
+            0, 49152, 40960, 36864, 34816,
+            33792, 33280, 33024, 32896, 32832,
+            33728, 37376, 43264, 51456, 1152,
+            1152, 576, 576, 448, 0
+        };
+
+    static const uint16_t const mousePointerWhite[] =
+        {
+            0, 0, 16384, 24576, 28672,
+            30720, 31744, 32256, 32512, 32640,
+            31744, 27648, 17920, 1536, 768,
+            768, 384, 384, 0, 0
+        };
+
+    int x_off = s->x_off;
+    int y_off = s->y_off;
+    int width = s->width;
+    int height = s->height;
+
+    if (   (*x - x_off) >= 0 && *x < (width + x_off)
+           && (*y - y_off) >= 0 && *y < (height + y_off) ) {
+        int line;
+        uint8_t *im_data = (uint8_t*)image->data;
+
+        im_data += (image->bytes_per_line * (*y - y_off)); // shift to right line
+        im_data += (image->bits_per_pixel / 8 * (*x - x_off)); // shift to right pixel
+
+        switch(image->bits_per_pixel) {
+        case 32:
+            DRAW_CURSOR_TEMPLATE(uint32_t);
+            break;
+        case 16:
+            DRAW_CURSOR_TEMPLATE(uint16_t);
+            break;
+        default:
+            /* XXX: How do we deal with 24bit and 8bit modes ? */
+            break;
+        }
+    }
 }
 
 
@@ -325,127 +325,127 @@
 static int
 XGetZPixmap(Display *dpy, Drawable d, XImage *image, int x, int y)
 {
-	xGetImageReply rep;
-	xGetImageReq *req;
-	long nbytes;
-  
-	if (!image) {
-		return False;
-	}
-
-	LockDisplay(dpy);
-	GetReq(GetImage, req);
-
-	/* First set up the standard stuff in the request */
-	req->drawable = d;
-	req->x = x;
-	req->y = y;
-	req->width = image->width;
-	req->height = image->height;
-	req->planeMask = (unsigned int)AllPlanes;
-	req->format = ZPixmap;
-  
-	if (!_XReply(dpy, (xReply *) &rep, 0, xFalse) || !rep.length) {
-		UnlockDisplay(dpy);
-		SyncHandle();
-		return False;
-	}
-  
-	nbytes = (long)rep.length << 2;
-	_XReadPad(dpy, image->data, nbytes);
-  
-	UnlockDisplay(dpy);
-	SyncHandle();
-	return True;
+    xGetImageReply rep;
+    xGetImageReq *req;
+    long nbytes;
+
+    if (!image) {
+        return False;
+    }
+
+    LockDisplay(dpy);
+    GetReq(GetImage, req);
+
+    /* First set up the standard stuff in the request */
+    req->drawable = d;
+    req->x = x;
+    req->y = y;
+    req->width = image->width;
+    req->height = image->height;
+    req->planeMask = (unsigned int)AllPlanes;
+    req->format = ZPixmap;
+
+    if (!_XReply(dpy, (xReply *) &rep, 0, xFalse) || !rep.length) {
+        UnlockDisplay(dpy);
+        SyncHandle();
+        return False;
+    }
+
+    nbytes = (long)rep.length << 2;
+    _XReadPad(dpy, image->data, nbytes);
+
+    UnlockDisplay(dpy);
+    SyncHandle();
+    return True;
 }
 
 static int x11grab_read_packet(AVFormatContext *s1, AVPacket *pkt)
 {
-	X11Grab *s = s1->priv_data;
-	Display *dpy = s->dpy;
-	XImage *image = s->image;
-	int x_off = s->x_off;
-	int y_off = s->y_off;
-
-	int64_t curtime, delay;
-	struct timespec ts;
-  
-	/* Calculate the time of the next frame */
-	s->time_frame += int64_t_C(1000000);
-  
-	/* wait based on the frame rate */
-	for(;;) {
-		curtime = av_gettime();
-		delay = s->time_frame  * s->frame_rate_base / s->frame_rate - curtime;
-		if (delay <= 0) {
-			if (delay < int64_t_C(-1000000) * s->frame_rate_base / s->frame_rate) {
-				s->time_frame += int64_t_C(1000000);
-			}
-			break;
-		}    
-		ts.tv_sec = delay / 1000000;
-		ts.tv_nsec = (delay % 1000000) * 1000;
-		nanosleep(&ts, NULL);
-	}
-  
-	if (av_new_packet(pkt, s->frame_size) < 0) {
-		return AVERROR_IO;
-	}
-  
-	pkt->pts = curtime & ((1LL << 48) - 1);
-  
-	if(s->use_shm) {
-		if (!XShmGetImage(dpy, RootWindow(dpy, DefaultScreen(dpy)), image, x_off, y_off, AllPlanes)) {
-			av_log (s1, AV_LOG_INFO, "XShmGetImage() failed\n");
-		}
-	} else {
-		if (!XGetZPixmap(dpy, RootWindow(dpy, DefaultScreen(dpy)), image, x_off, y_off)) {
-			av_log (s1, AV_LOG_INFO, "XGetZPixmap() failed\n");
-		}
-	}
-
-	{
-		int pointer_x, pointer_y;
-		getCurrentPointer(s1, s, &pointer_x, &pointer_y);
-		paintMousePointer(s1, s, &pointer_x, &pointer_y, image);
-	}
-
-
-	/* XXX: avoid memcpy */
-	memcpy(pkt->data, image->data, s->frame_size);
-	return s->frame_size;
+    X11Grab *s = s1->priv_data;
+    Display *dpy = s->dpy;
+    XImage *image = s->image;
+    int x_off = s->x_off;
+    int y_off = s->y_off;
+
+    int64_t curtime, delay;
+    struct timespec ts;
+
+    /* Calculate the time of the next frame */
+    s->time_frame += int64_t_C(1000000);
+
+    /* wait based on the frame rate */
+    for(;;) {
+        curtime = av_gettime();
+        delay = s->time_frame  * s->frame_rate_base / s->frame_rate - curtime;
+        if (delay <= 0) {
+            if (delay < int64_t_C(-1000000) * s->frame_rate_base / s->frame_rate) {
+                s->time_frame += int64_t_C(1000000);
+            }
+            break;
+        }
+        ts.tv_sec = delay / 1000000;
+        ts.tv_nsec = (delay % 1000000) * 1000;
+        nanosleep(&ts, NULL);
+    }
+
+    if (av_new_packet(pkt, s->frame_size) < 0) {
+        return AVERROR_IO;
+    }
+
+    pkt->pts = curtime & ((1LL << 48) - 1);
+
+    if(s->use_shm) {
+        if (!XShmGetImage(dpy, RootWindow(dpy, DefaultScreen(dpy)), image, x_off, y_off, AllPlanes)) {
+            av_log (s1, AV_LOG_INFO, "XShmGetImage() failed\n");
+        }
+    } else {
+        if (!XGetZPixmap(dpy, RootWindow(dpy, DefaultScreen(dpy)), image, x_off, y_off)) {
+            av_log (s1, AV_LOG_INFO, "XGetZPixmap() failed\n");
+        }
+    }
+
+    {
+        int pointer_x, pointer_y;
+        getCurrentPointer(s1, s, &pointer_x, &pointer_y);
+        paintMousePointer(s1, s, &pointer_x, &pointer_y, image);
+    }
+
+
+    /* XXX: avoid memcpy */
+    memcpy(pkt->data, image->data, s->frame_size);
+    return s->frame_size;
 }
 
 static int x11grab_read_close(AVFormatContext *s1)
 {
-	X11Grab *x11grab = s1->priv_data;
+    X11Grab *x11grab = s1->priv_data;
 
-	/* Detach cleanly from shared mem */
-	if (x11grab->use_shm) {
-		XShmDetach(x11grab->dpy, &x11grab->shminfo);
-		shmdt(x11grab->shminfo.shmaddr);
-		shmctl(x11grab->shminfo.shmid, IPC_RMID, NULL);
-	}
-
-	/* Destroy X11 image */
-	if (x11grab->image) {
-		XDestroyImage(x11grab->image);
-		x11grab->image = NULL;
-	}
-
-	/* Free X11 display */
-	XCloseDisplay(x11grab->dpy);
-	return 0;
+    /* Detach cleanly from shared mem */
+    if (x11grab->use_shm) {
+        XShmDetach(x11grab->dpy, &x11grab->shminfo);
+        shmdt(x11grab->shminfo.shmaddr);
+        shmctl(x11grab->shminfo.shmid, IPC_RMID, NULL);
+    }
+
+    /* Destroy X11 image */
+    if (x11grab->image) {
+        XDestroyImage(x11grab->image);
+        x11grab->image = NULL;
+    }
+
+    /* Free X11 display */
+    XCloseDisplay(x11grab->dpy);
+    return 0;
 }
 
 AVInputFormat x11_grab_device_demuxer =
 {
-	"x11grab",
-	"X11grab",
-	sizeof(X11Grab),
-	NULL,
-	x11grab_read_header,
-	x11grab_read_packet,
-	x11grab_read_close,
-	.flags = AVFMT_NOFILE,
+    "x11grab",
+    "X11grab",
+    sizeof(X11Grab),
+    NULL,
+    x11grab_read_header,
+    x11grab_read_packet,
+    x11grab_read_close,
+    .flags = AVFMT_NOFILE,
 };




More information about the ffmpeg-cvslog mailing list