Changeset 10844

Timestamp:

Jun 25, 2010, 1:13:35 AM (12 years ago)

Author:

charles

Message:

(trunk) #3311 "MingW build of Transmission" -- modify tr_strip_positional_args() to strip out the ' printf flag. Platforms not supporting $ tend to not support ' either

Location:

trunk/libtransmission

Files:

• utils-test.c (modified) (2 diffs)
• utils.c (modified) (4 diffs)
• utils.h (modified) (2 diffs)

trunk/libtransmission/utils-test.c

@@ -104 +104 @@
 
 static int
+test_strip_positional_args( void )
+{
+    const char * in;
+    const char * out;
+    const char * expected;
+
+    in = "Hello %1$s foo %2$.*f";
+    expected = "Hello %s foo %.*f";
+    out = tr_strip_positional_args( in );
+    check( out != NULL )
+    check( !strcmp( out, expected ) )
+
+    in = "Hello %1$'d foo %2$'f";
+    expected = "Hello %d foo %f";
+    out = tr_strip_positional_args( in );
+    check( out != NULL )
+    check( !strcmp( out, expected ) )
+
+    return 0;
+}
+
+static int
 test_strstrip( void )
 {
@@ -393 +415 @@
     if( ( i = test_lowerbound( ) ) )
         return i;
+    if( ( i = test_strip_positional_args( ) ) )
+        return i;
     if( ( i = test_strstrip( ) ) )
         return i;

trunk/libtransmission/utils.c

@@ -405 +405 @@
 ***/
 
-#ifdef DISABLE_GETTEXT
-
 const char*
 tr_strip_positional_args( const char* str )
@@ -425 +423 @@
     {
         *out++ = *str;
+
         if( ( *str == '%' ) && isdigit( str[1] ) )
         {
@@ -430 +429 @@
             while( isdigit( *tmp ) )
                 ++tmp;
-
             if( *tmp == '$' )
-                str = tmp;
+                str = tmp[1]=='\'' ? tmp+1 : tmp;
         }
+
+        if( ( *str == '%' ) && ( str[1] == '\'' ) )
+            str = str + 1;
+ 
     }
     *out = '\0';
@@ -439 +441 @@
     return strcmp( buf, in ) ? buf : in;
 }
-
-#endif
 
 /**

trunk/libtransmission/utils.h

@@ -89 +89 @@
 ***/
 
+const char * tr_strip_positional_args( const char * fmt );
+
 #if !defined( _ )
  #if defined( HAVE_LIBINTL_H ) && !defined( SYS_DARWIN )
@@ -105 +107 @@
 #endif
 #ifdef DISABLE_GETTEXT
- const char * tr_strip_positional_args( const char * fmt );
  #undef _
  #define _( a ) tr_strip_positional_args( a )