diff libvo/vo_png.c @ 24671:fbcd9dcb0daf

Get rid of mp_msg_test in vo_png, only reason to use it is performance and that is not critical here and the way it was used probably would not improve performance anyway

--- a/libvo/vo_png.c	Wed Oct 03 11:51:53 2007 +0000
+++ b/libvo/vo_png.c	Wed Oct 03 11:55:30 2007 +0000
@@ -52,8 +52,7 @@
  		    mp_msg(MSGT_VO,MSGL_INFO, MSGTR_LIBVO_PNG_Warning3);
 	    }	    
     
-    if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-        mp_msg(MSGT_VO,MSGL_DBG2, "PNG Compression level %i\n", z_compression); }
+    mp_msg(MSGT_VO,MSGL_DBG2, "PNG Compression level %i\n", z_compression);
 	  	
     return 0;
 }
@@ -73,15 +72,13 @@
     png.info_ptr = png_create_info_struct(png.png_ptr);
    
     if (!png.png_ptr) {
-       if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-           mp_msg(MSGT_VO,MSGL_DBG2, "PNG Failed to init png pointer\n"); }
+       mp_msg(MSGT_VO,MSGL_DBG2, "PNG Failed to init png pointer\n");
        png.status = ERROR;
        return png;
     }   
     
     if (!png.info_ptr) {
-       if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-           mp_msg(MSGT_VO,MSGL_DBG2, "PNG Failed to init png infopointer\n"); }
+       mp_msg(MSGT_VO,MSGL_DBG2, "PNG Failed to init png infopointer\n");
        png_destroy_write_struct(&png.png_ptr,
          (png_infopp)NULL);
        png.status = ERROR;
@@ -89,8 +86,7 @@
     }
     
     if (setjmp(png.png_ptr->jmpbuf)) {
-	if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-            mp_msg(MSGT_VO,MSGL_DBG2, "PNG Internal error!\n"); }
+        mp_msg(MSGT_VO,MSGL_DBG2, "PNG Internal error!\n");
         png_destroy_write_struct(&png.png_ptr, &png.info_ptr);
         fclose(png.fp);
         png.status = ERROR;
@@ -104,8 +100,7 @@
        	return png;
     }	    
     
-    if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-        mp_msg(MSGT_VO,MSGL_DBG2, "PNG Init IO\n"); }
+    mp_msg(MSGT_VO,MSGL_DBG2, "PNG Init IO\n");
     png_init_io(png.png_ptr, png.fp);
 
     /* set the zlib compression level */
@@ -119,13 +114,11 @@
        8, PNG_COLOR_TYPE_RGB, PNG_INTERLACE_NONE,
        PNG_COMPRESSION_TYPE_DEFAULT, PNG_FILTER_TYPE_DEFAULT);
     
-    if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-        mp_msg(MSGT_VO,MSGL_DBG2, "PNG Write Info\n"); }
+    mp_msg(MSGT_VO,MSGL_DBG2, "PNG Write Info\n");
     png_write_info(png.png_ptr, png.info_ptr);
     
     if(swapped) {
-    	if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-            mp_msg(MSGT_VO,MSGL_DBG2, "PNG Set BGR Conversion\n"); }
+        mp_msg(MSGT_VO,MSGL_DBG2, "PNG Set BGR Conversion\n");
     	png_set_bgr(png.png_ptr);
     }	
 
@@ -135,12 +128,10 @@
        
 static uint8_t destroy_png(struct pngdata png) {
 	    
-    if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-        mp_msg(MSGT_VO,MSGL_DBG2, "PNG Write End\n"); }
+    mp_msg(MSGT_VO,MSGL_DBG2, "PNG Write End\n");
     png_write_end(png.png_ptr, png.info_ptr);
 
-    if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-        mp_msg(MSGT_VO,MSGL_DBG2, "PNG Destroy Write Struct\n"); }
+    mp_msg(MSGT_VO,MSGL_DBG2, "PNG Destroy Write Struct\n");
     png_destroy_write_struct(&png.png_ptr, &png.info_ptr);
     
     fclose (png.fp);
@@ -166,8 +157,7 @@
 	    return 1;
     }	     
 
-    if( mp_msg_test(MSGT_VO,MSGL_DBG2) ) {
-        mp_msg(MSGT_VO,MSGL_DBG2, "PNG Creating Row Pointers\n"); }
+    mp_msg(MSGT_VO,MSGL_DBG2, "PNG Creating Row Pointers\n");
     for ( k = 0; k < mpi->h; k++ )
 	row_pointers[k] = mpi->planes[0]+mpi->stride[0]*k;