]> granicus.if.org Git - transmission/commitdiff
code cleanup
authorCharles Kerr <charles@transmissionbt.com>
Wed, 9 Jul 2008 15:31:35 +0000 (15:31 +0000)
committerCharles Kerr <charles@transmissionbt.com>
Wed, 9 Jul 2008 15:31:35 +0000 (15:31 +0000)
cli/cli.c
libtransmission/tr-getopt.c
libtransmission/tr-getopt.h

index 92ce736bc990f79d450d2cb0c360ab7fbb4a865d..f2874b4119f5cbb886b9dfa0cde449372902c216 100644 (file)
--- a/cli/cli.c
+++ b/cli/cli.c
@@ -54,8 +54,8 @@ static int           verify           = 0;
 static sig_atomic_t  gotsig           = 0;
 static sig_atomic_t  manualUpdate     = 0;
 
-static const char   * torrentPath   = NULL;
-static const char   * downloadDir   = NULL;
+static const char   * torrentPath  = NULL;
+static const char   * downloadDir  = NULL;
 static const char   * finishCall   = NULL;
 static const char   * announce     = NULL;
 static const char   * configdir    = NULL;
@@ -165,16 +165,17 @@ dumpInfo( FILE * out, const tr_info * inf )
                   inf->totalSize, inf->totalSize / inf->pieceSize,
                   inf->pieceSize, inf->totalSize % inf->pieceSize );
 
-    if( inf->comment[0] )
+    if( inf->comment && *inf->comment )
         fprintf( out, "comment:\t%s\n", inf->comment );
-    if( inf->creator[0] )
+    if( inf->creator && *inf->creator )
         fprintf( out, "creator:\t%s\n", inf->creator );
     if( inf->isPrivate )
         fprintf( out, "private flag set\n" );
 
     fprintf( out, "file(s):\n" );
     for( ff=0; ff<inf->fileCount; ++ff )
-        fprintf( out, "\t%s (%"PRIu64")\n", inf->files[ff].name, inf->files[ff].length );
+        fprintf( out, "\t%s (%"PRIu64")\n", inf->files[ff].name,
+                                            inf->files[ff].length );
 }
 
 static void
@@ -213,10 +214,7 @@ getStatusStr( const tr_stat * st, char * buf, size_t buflen )
                   st->peersGettingFromUs, st->peersConnected,
                   st->rateUpload, ratioStr );
     }
-    else if( st->status & TR_STATUS_STOPPED )
-    {
-        *buf = '\0';
-    }
+    else *buf = '\0';
 }
 
 int
@@ -296,17 +294,17 @@ main( int argc, char ** argv )
     if( sourceFile && *sourceFile ) /* creating a torrent */
     {
         int err;
-        tr_metainfo_builder * builder = tr_metaInfoBuilderCreate( h, sourceFile );
+        tr_metainfo_builder * b = tr_metaInfoBuilderCreate( h, sourceFile );
         tr_tracker_info ti;
         ti.tier = 0;
         ti.announce = (char*) announce;
-        tr_makeMetaInfo( builder, torrentPath, &ti, 1, comment, isPrivate );
-        while( !builder->isDone ) {
+        tr_makeMetaInfo( b, torrentPath, &ti, 1, comment, isPrivate );
+        while( !b->isDone ) {
             tr_wait( 1000 );
             printf( "." );
         }
-        err = builder->result;
-        tr_metaInfoBuilderFree( builder );
+        err = b->result;
+        tr_metaInfoBuilderFree( b );
         return err;
     }
 
@@ -338,6 +336,7 @@ main( int argc, char ** argv )
                     tr_httpParseURL( scrape, -1, &host, NULL, NULL );
                     ++leftToScrape;
                     tr_webRun( h, url, NULL, scrapeDoneFunc, host );
+                    tr_free( host );
                     tr_free( url );
                 }
             }
@@ -393,16 +392,16 @@ main( int argc, char ** argv )
 
         if( gotsig ) {
             gotsig = 0;
-fprintf( stderr, "stopping torrent...\n" );
+            printf( "stopping torrent...\n" );
             tr_torrentStop( tor );
         }
         
         if( manualUpdate ) {
             manualUpdate = 0;
             if ( !tr_torrentCanManualUpdate( tor ) )
-                fprintf( stderr, "\rReceived SIGHUP, but can't send a manual update now\n" );
+                fprintf( stderr, "\nReceived SIGHUP, but can't send a manual update now\n" );
             else {
-                fprintf( stderr, "\rReceived SIGHUP: manual update scheduled\n" );
+                fprintf( stderr, "\nReceived SIGHUP: manual update scheduled\n" );
                 tr_torrentManualUpdate( tor );
             }
         }
@@ -412,9 +411,9 @@ fprintf( stderr, "stopping torrent...\n" );
             break;
 
         getStatusStr( st, line, sizeof( line ) );
-        printf( "%-*s\n", LINEWIDTH, line );
+        printf( "\r%-*s", LINEWIDTH, line );
         if( st->error )
-            printf( "%s\n", st->errorString );
+            printf( "\n%s\n", st->errorString );
     }
 
 cleanup:
index 626b4f09fdcc483a9ba6a77ac901723e5e4239f2..fc394d3c2981e529de66602badd7865adcab2787 100644 (file)
@@ -1,11 +1,23 @@
 /*
  * This file Copyright (C) 2008 Charles Kerr <charles@rebelbase.com>
  *
- * This file is licensed by the GPL version 2.  Works owned by the
- * Transmission project are granted a special exemption to clause 2(b)
- * so that the bulk of its code can remain under the MIT license. 
- * This exemption does not extend to derived works not owned by
- * the Transmission project.
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
  *
  * $Id:$
  */
 #define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
-int option_index = 1;
+int tr_optind = 1;
 
 static const char*
 getArgName( const tr_option * opt )
 {
-    if( !opt->has_arg )   return "";
-    if( opt->argName ) return opt->argName;
-    return "<args>";
+    char * arg;
+
+    if( !opt->has_arg )
+        arg = "";
+    else if( opt->argName )
+        arg = opt->argName;
+    else
+        arg = "<args>";
+
+    return arg;
 }
 
 static void
@@ -43,46 +62,50 @@ getopts_usage_line( const tr_option * opt,
                                          opt->description );
 }
 
-void
-tr_getopt_usage( const char             * progName,
-                 const char             * description,
-                 const struct tr_option    opts[] )
+static void
+maxWidth( const struct tr_option * o,
+          int * longWidth, int * shortWidth, int * argWidth )
 {
-  int count;
-  int longWidth = 0;
-  int shortWidth = 0;
-  int argWidth = 0;
-  struct tr_option help;
-
-  for( count=0; opts[count].description; ++count )
-  {
     const char * arg;
 
-    if( opts[count].longName ) 
-      longWidth = MAX( longWidth, (int)strlen( opts[count].longName ) );
+    if( o->longName ) 
+        *longWidth = MAX( *longWidth, (int)strlen( o->longName ) );
 
-    if( opts[count].shortName )
-      shortWidth = MAX( shortWidth, (int)strlen( opts[count].shortName ) );
+    if( o->shortName )
+        *shortWidth = MAX( *shortWidth, (int)strlen( o->shortName ) );
 
-    if(( arg = getArgName( &opts[count] )))
-      argWidth = MAX( argWidth, (int)strlen( arg ) );
-  }
+    if(( arg = getArgName( o )))
+        *argWidth = MAX( *argWidth, (int)strlen( arg ) );
+}
 
-  if( !description )
-    description = "Usage: %s [options]";
-  printf( description, progName );
-  printf( "\n\n" );
-  printf( "Options:\n" );
+void
+tr_getopt_usage( const char              * progName,
+                 const char              * description,
+                 const struct tr_option    opts[] )
+{
+    int longWidth = 0;
+    int shortWidth = 0;
+    int argWidth = 0;
+    struct tr_option help;
+    const struct tr_option * o;
 
-  help.val = -1;
-  help.longName = "help";
-  help.description = "Display this help page and exit";
-  help.shortName = "h";
-  help.has_arg = 0;
-  getopts_usage_line( &help, longWidth, shortWidth, argWidth );
-  
-  for( count=0; opts[count].description; ++count )
-      getopts_usage_line( &opts[count], longWidth, shortWidth, argWidth );
+    for( o=opts; o->val; ++o )
+        maxWidth( o, &longWidth, &shortWidth, &argWidth );
+
+    help.val = -1;
+    help.longName = "help";
+    help.description = "Display this help page and exit";
+    help.shortName = "h";
+    help.has_arg = 0;
+    maxWidth( &help, &longWidth, &shortWidth, &argWidth );
+
+    if( description == NULL )
+        description = "Usage: %s [options]";
+    printf( description, progName );
+    printf( "\n\nOptions:\n" );
+    getopts_usage_line( &help, longWidth, shortWidth, argWidth );
+    for( o=opts; o->val; ++o )
+        getopts_usage_line( o, longWidth, shortWidth, argWidth );
 }
 
 static const tr_option *
@@ -123,16 +146,6 @@ findOption( const tr_option   * opts,
     return NULL;
 }
 
-static void
-showUsageAndExit( const char        * appName,
-                  const char        * description,
-                  const tr_option   * opts )
-{
-    tr_getopt_usage( appName, description, opts );
-    exit( 0 );
-}
-
-
 int
 tr_getopt( const char        * usage,
            int                 argc,
@@ -145,23 +158,23 @@ tr_getopt( const char        * usage,
     const tr_option * o = NULL;
 
     *setme_optarg = NULL;  
-
-    if( argc==1 || argc==option_index )
-        return TR_OPT_DONE;
   
     /* handle the builtin 'help' option */
-    for( i=1; i<argc; ++i )
-        if( !strcmp(argv[i], "-h") || !strcmp(argv[i], "--help" ) )
-            showUsageAndExit( argv[0], usage, opts );
+    for( i=1; i<argc; ++i ) {
+        if( !strcmp(argv[i], "-h") || !strcmp(argv[i], "--help" ) ) {
+            tr_getopt_usage( argv[0], usage, opts );
+            exit( 0 );
+        }
+    }
 
-    /* out of options */
-    if( option_index >= argc )
+    /* out of options? */
+    if( argc==1 || tr_optind>=argc )
         return TR_OPT_DONE;
 
-    o = findOption( opts, argv[option_index], &nest );
+    o = findOption( opts, argv[tr_optind], &nest );
     if( !o ) {
         /* let the user know we got an unknown option... */
-        *setme_optarg = argv[option_index++];
+        *setme_optarg = argv[tr_optind++];
         return TR_OPT_UNK;
     }
 
@@ -170,23 +183,23 @@ tr_getopt( const char        * usage,
         if( nest )
             return TR_OPT_ERR;
         *setme_optarg = NULL;
-        option_index++;
+        tr_optind++;
         return o->val;
     }
 
     /* option needed an argument, and it was nested in this string */
     if( nest ) {
         *setme_optarg = nest;
-        option_index++;
+        tr_optind++;
         return o->val;
     }
 
     /* throw an error if the option needed an argument but didn't get one */
-    if( ++option_index >= argc )
+    if( ++tr_optind >= argc )
         return TR_OPT_ERR;
-    if( findOption( opts, argv[option_index], NULL ))
+    if( findOption( opts, argv[tr_optind], NULL ))
         return TR_OPT_ERR;
 
-    *setme_optarg = argv[option_index++];
+    *setme_optarg = argv[tr_optind++];
     return o->val;
 }
index 4faa8e92c9ed7dc1141b786daad1ceebe3294420..e531901d0610bc2703ba1d00f8ec870413cfe13c 100644 (file)
@@ -1,11 +1,23 @@
 /*
  * This file Copyright (C) 2008 Charles Kerr <charles@rebelbase.com>
  *
- * This file is licensed by the GPL version 2.  Works owned by the
- * Transmission project are granted a special exemption to clause 2(b)
- * so that the bulk of its code can remain under the MIT license. 
- * This exemption does not extend to derived works not owned by
- * the Transmission project.
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
  *
  * $Id:$
  */
@@ -41,7 +53,7 @@ enum
 
 /**
  * @return TR_GETOPT_DONE, TR_GETOPT_ERR, TR_GETOPT_UNK,
- *         or the matching tr_option's `value' field
+ *         or the matching tr_option's `val' field
  */
 int tr_getopt( const char        * summary,
                int                 argc,