[FFmpeg-cvslog] r23158 - in trunk: libavcodec/internal.h libavcodec/mpegvideo.c libavcodec/utils.c libavformat/utils.c

[cehoyos]

Author: cehoyos
Date: Tue May 18 21:16:40 2010
New Revision: 23158

Log:
Factorize some code into the new function ff_toupper4().

Patch by Francesco Lavra, francescolavra interfree it

Modified:
   trunk/libavcodec/internal.h
   trunk/libavcodec/mpegvideo.c
   trunk/libavcodec/utils.c
   trunk/libavformat/utils.c

Modified: trunk/libavcodec/internal.h
==============================================================================
--- trunk/libavcodec/internal.h	Tue May 18 19:28:07 2010	(r23157)
+++ trunk/libavcodec/internal.h	Tue May 18 21:16:40 2010	(r23158)
@@ -48,4 +48,6 @@ AVHWAccel *ff_find_hwaccel(enum CodecID 
  */
 int ff_match_2uint16(const uint16_t (*tab)[2], int size, int a, int b);
 
+unsigned int ff_toupper4(unsigned int x);
+
 #endif /* AVCODEC_INTERNAL_H */

Modified: trunk/libavcodec/mpegvideo.c
==============================================================================
--- trunk/libavcodec/mpegvideo.c	Tue May 18 19:28:07 2010	(r23157)
+++ trunk/libavcodec/mpegvideo.c	Tue May 18 21:16:40 2010	(r23158)
@@ -30,6 +30,7 @@
 #include "libavutil/intmath.h"
 #include "avcodec.h"
 #include "dsputil.h"
+#include "internal.h"
 #include "mpegvideo.h"
 #include "mpegvideo_common.h"
 #include "mjpegenc.h"
@@ -530,15 +531,9 @@ av_cold int MPV_common_init(MpegEncConte
     yc_size = y_size + 2 * c_size;
 
     /* convert fourcc to upper case */
-    s->codec_tag=          toupper( s->avctx->codec_tag     &0xFF)
-                        + (toupper((s->avctx->codec_tag>>8 )&0xFF)<<8 )
-                        + (toupper((s->avctx->codec_tag>>16)&0xFF)<<16)
-                        + (toupper((s->avctx->codec_tag>>24)&0xFF)<<24);
+    s->codec_tag = ff_toupper4(s->avctx->codec_tag);
 
-    s->stream_codec_tag=          toupper( s->avctx->stream_codec_tag     &0xFF)
-                               + (toupper((s->avctx->stream_codec_tag>>8 )&0xFF)<<8 )
-                               + (toupper((s->avctx->stream_codec_tag>>16)&0xFF)<<16)
-                               + (toupper((s->avctx->stream_codec_tag>>24)&0xFF)<<24);
+    s->stream_codec_tag = ff_toupper4(s->avctx->stream_codec_tag);
 
     s->avctx->coded_frame= (AVFrame*)&s->current_picture;
 

Modified: trunk/libavcodec/utils.c
==============================================================================
--- trunk/libavcodec/utils.c	Tue May 18 19:28:07 2010	(r23157)
+++ trunk/libavcodec/utils.c	Tue May 18 21:16:40 2010	(r23158)
@@ -1294,3 +1294,11 @@ int av_lockmgr_register(int (*cb)(void *
     }
     return 0;
 }
+
+unsigned int ff_toupper4(unsigned int x)
+{
+    return     toupper( x     &0xFF)
+            + (toupper((x>>8 )&0xFF)<<8 )
+            + (toupper((x>>16)&0xFF)<<16)
+            + (toupper((x>>24)&0xFF)<<24);
+}

Modified: trunk/libavformat/utils.c
==============================================================================
--- trunk/libavformat/utils.c	Tue May 18 19:28:07 2010	(r23157)
+++ trunk/libavformat/utils.c	Tue May 18 21:16:40 2010	(r23158)
@@ -20,6 +20,7 @@
  */
 #include "avformat.h"
 #include "internal.h"
+#include "libavcodec/internal.h"
 #include "libavcodec/opt.h"
 #include "metadata.h"
 #include "libavutil/avstring.h"
@@ -2041,10 +2042,7 @@ enum CodecID ff_codec_get_id(const AVCod
             return tags[i].id;
     }
     for(i=0; tags[i].id != CODEC_ID_NONE; i++) {
-        if(   toupper((tag >> 0)&0xFF) == toupper((tags[i].tag >> 0)&0xFF)
-           && toupper((tag >> 8)&0xFF) == toupper((tags[i].tag >> 8)&0xFF)
-           && toupper((tag >>16)&0xFF) == toupper((tags[i].tag >>16)&0xFF)
-           && toupper((tag >>24)&0xFF) == toupper((tags[i].tag >>24)&0xFF))
+        if (ff_toupper4(tag) == ff_toupper4(tags[i].tag))
             return tags[i].id;
     }
     return CODEC_ID_NONE;