Changeset 13790


Ignore:
Timestamp:
Jan 15, 2013, 6:10:16 PM (8 years ago)
Author:
jordan
Message:

(daemon) #5225 nightly regression: remote option '-l' displays warning -- fixed.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/daemon/remote.c

    r13736 r13790  
    553553
    554554static void
    555 addIdArg (tr_variant * args, const char * id)
    556 {
    557   if (!*id)
    558     {
    559       fprintf (stderr, "No torrent specified!  Please use the -t option first.\n");
    560       id = "-1"; /* no torrent will have this ID, so should be a no-op */
    561     }
    562   else if (!tr_strcmp0 (id, "active"))
     555addIdArg (tr_variant * args, const char * id, const char * fallback)
     556{
     557  if (!id || !*id)
     558    {
     559      id = fallback;
     560
     561      if (!id || !*id)
     562        {
     563          fprintf (stderr, "No torrent specified!  Please use the -t option first.\n");
     564          id = "-1"; /* no torrent will have this ID, so will act as a no-op */
     565        }
     566    }
     567
     568  if (!tr_strcmp0 (id, "active"))
    563569    {
    564570      tr_variantDictAddStr (args, TR_KEY_ids, "recently-active");
    … …  
    18951901                    if (sset != 0) status |= flush (rpcurl, &sset);
    18961902                    if (tadd != 0) status |= flush (rpcurl, &tadd);
    1897                     if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id); status |= flush (rpcurl, &tset); }
     1903                    if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id, NULL); status |= flush (rpcurl, &tset); }
    18981904                    tadd = tr_new0 (tr_variant, 1);
    18991905                    tr_variantInitDict (tadd, 3);
    … …  
    19321938                case 't': /* set current torrent */
    19331939                    if (tadd != 0) status |= flush (rpcurl, &tadd);
    1934                     if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id); status |= flush (rpcurl, &tset); }
     1940                    if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id, NULL); status |= flush (rpcurl, &tset); }
    19351941                    tr_strlcpy (id, optarg, sizeof (id));
    19361942                    break;
    … …  
    19741980            fields = tr_variantDictAddList (args, TR_KEY_fields, 0);
    19751981
    1976             if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id); status |= flush (rpcurl, &tset); }
     1982            if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id, NULL); status |= flush (rpcurl, &tset); }
    19771983
    19781984            switch (c)
    … …  
    19811987                          n = TR_N_ELEMENTS (details_keys);
    19821988                          for (i=0; i<n; ++i) tr_variantListAddQuark (fields, details_keys[i]);
    1983                           addIdArg (args, id);
     1989                          addIdArg (args, id, NULL);
    19841990                          break;
    19851991                case 'l': tr_variantDictAddInt (top, TR_KEY_tag, TAG_LIST);
    19861992                          n = TR_N_ELEMENTS (list_keys);
    19871993                          for (i=0; i<n; ++i) tr_variantListAddQuark (fields, list_keys[i]);
    1988                           addIdArg (args, id);
     1994                          addIdArg (args, id, "all");
    19891995                          break;
    19901996                case 940: tr_variantDictAddInt (top, TR_KEY_tag, TAG_FILES);
    19911997                          n = TR_N_ELEMENTS (files_keys);
    19921998                          for (i=0; i<n; ++i) tr_variantListAddQuark (fields, files_keys[i]);
    1993                           addIdArg (args, id);
     1999                          addIdArg (args, id, NULL);
    19942000                          break;
    19952001                case 941: tr_variantDictAddInt (top, TR_KEY_tag, TAG_PEERS);
    19962002                          tr_variantListAddStr (fields, "peers");
    1997                           addIdArg (args, id);
     2003                          addIdArg (args, id, NULL);
    19982004                          break;
    19992005                case 942: tr_variantDictAddInt (top, TR_KEY_tag, TAG_PIECES);
    20002006                          tr_variantListAddStr (fields, "pieces");
    20012007                          tr_variantListAddStr (fields, "pieceCount");
    2002                           addIdArg (args, id);
     2008                          addIdArg (args, id, NULL);
    20032009                          break;
    20042010                case 943: tr_variantDictAddInt (top, TR_KEY_tag, TAG_TRACKERS);
    20052011                          tr_variantListAddStr (fields, "trackerStats");
    2006                           addIdArg (args, id);
     2012                          addIdArg (args, id, NULL);
    20072013                          break;
    20082014                default:  assert ("unhandled value" && 0);
    … …  
    22132219                tr_variantDictAddStr (args, TR_KEY_location, optarg);
    22142220                tr_variantDictAddBool (args, TR_KEY_move, false);
    2215                 addIdArg (args, id);
     2221                addIdArg (args, id, NULL);
    22162222                status |= flush (rpcurl, &top);
    22172223                break;
    … …  
    22372243                    tr_variantInitDict (top, 2);
    22382244                    tr_variantDictAddStr (top, TR_KEY_method, "torrent-start");
    2239                     addIdArg (tr_variantDictAddDict (top, ARGUMENTS, 1), id);
     2245                    addIdArg (tr_variantDictAddDict (top, ARGUMENTS, 1), id, NULL);
    22402246                    status |= flush (rpcurl, &top);
    22412247                }
    … …  
    22502256                    tr_variantInitDict (top, 2);
    22512257                    tr_variantDictAddStr (top, TR_KEY_method, "torrent-stop");
    2252                     addIdArg (tr_variantDictAddDict (top, ARGUMENTS, 1), id);
     2258                    addIdArg (tr_variantDictAddDict (top, ARGUMENTS, 1), id, NULL);
    22532259                    status |= flush (rpcurl, &top);
    22542260                }
    … …  
    23042310            {
    23052311                tr_variant * top;
    2306                 if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id); status |= flush (rpcurl, &tset); }
     2312                if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id, NULL); status |= flush (rpcurl, &tset); }
    23072313                top = tr_new0 (tr_variant, 1);
    23082314                tr_variantInitDict (top, 2);
    23092315                tr_variantDictAddStr (top, TR_KEY_method, "torrent-reannounce");
    2310                 addIdArg (tr_variantDictAddDict (top, ARGUMENTS, 1), id);
     2316                addIdArg (tr_variantDictAddDict (top, ARGUMENTS, 1), id, NULL);
    23112317                status |= flush (rpcurl, &top);
    23122318                break;
    … …  
    23152321            {
    23162322                tr_variant * top;
    2317                 if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id); status |= flush (rpcurl, &tset); }
     2323                if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id, NULL); status |= flush (rpcurl, &tset); }
    23182324                top = tr_new0 (tr_variant, 1);
    23192325                tr_variantInitDict (top, 2);
    23202326                tr_variantDictAddStr (top, TR_KEY_method, "torrent-verify");
    2321                 addIdArg (tr_variantDictAddDict (top, ARGUMENTS, 1), id);
     2327                addIdArg (tr_variantDictAddDict (top, ARGUMENTS, 1), id, NULL);
    23222328                status |= flush (rpcurl, &top);
    23232329                break;
    … …  
    23322338                args = tr_variantDictAddDict (top, ARGUMENTS, 2);
    23332339                tr_variantDictAddBool (args, TR_KEY_delete_local_data, c=='R');
    2334                 addIdArg (args, id);
     2340                addIdArg (args, id, NULL);
    23352341                status |= flush (rpcurl, &top);
    23362342                break;
    … …  
    23452351                tr_variantDictAddStr (args, TR_KEY_location, optarg);
    23462352                tr_variantDictAddBool (args, TR_KEY_move, true);
    2347                 addIdArg (args, id);
     2353                addIdArg (args, id, NULL);
    23482354                status |= flush (rpcurl, &top);
    23492355                break;
    … …  
    23592365
    23602366    if (tadd != 0) status |= flush (rpcurl, &tadd);
    2361     if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id); status |= flush (rpcurl, &tset); }
     2367    if (tset != 0) { addIdArg (tr_variantDictFind (tset, ARGUMENTS), id, NULL); status |= flush (rpcurl, &tset); }
    23622368    if (sset != 0) status |= flush (rpcurl, &sset);
    23632369    return status;
Note: See TracChangeset for help on using the changeset viewer.