]> granicus.if.org Git - transmission/commitdiff
NSString literals
authorDmitry Serov <barbari100@gmail.com>
Sat, 8 Jul 2017 08:06:32 +0000 (15:06 +0700)
committerDmitry Serov <barbari100@gmail.com>
Sat, 8 Jul 2017 08:06:32 +0000 (15:06 +0700)
macosx/Controller.m
macosx/DragOverlayWindow.m
macosx/MessageWindowController.m
macosx/NSStringAdditions.m
macosx/PrefsController.m
macosx/Torrent.m
macosx/TrackerNode.m

index e47f409d696f6dd3abbcb71d0e30c3219dd1054e..82f78d9da18cbcfbbc6bcecc64a8aebfe8b1cbc8 100644 (file)
@@ -978,7 +978,7 @@ static void removeKeRangerRansomware()
         if (result != TR_PARSE_OK)
         {
             if (result == TR_PARSE_DUPLICATE)
-                [self duplicateOpenAlert: [NSString stringWithUTF8String: info.name]];
+                [self duplicateOpenAlert: @(info.name)];
             else if (result == TR_PARSE_ERR)
             {
                 if (type != ADD_AUTO)
@@ -1097,7 +1097,7 @@ static void removeKeRangerRansomware()
     if ((duplicateTorrent = tr_torrentFindFromMagnetLink(fLib, [address UTF8String])))
     {
         const tr_info * info = tr_torrentInfo(duplicateTorrent);
-        NSString * name = (info != NULL && info->name != NULL) ? [NSString stringWithUTF8String: info->name] : nil;
+        NSString * name = (info != NULL && info->name != NULL) ? @(info->name) : nil;
         [self duplicateOpenMagnetAlert: address transferName: name];
         return;
     }
@@ -4716,7 +4716,7 @@ static void removeKeRangerRansomware()
 {
     NSString * location = nil;
     if (tr_torrentGetDownloadDir(torrentStruct) != NULL)
-        location = [NSString stringWithUTF8String: tr_torrentGetDownloadDir(torrentStruct)];
+        location = @(tr_torrentGetDownloadDir(torrentStruct));
 
     Torrent * torrent = [[Torrent alloc] initWithTorrentStruct: torrentStruct location: location lib: fLib];
 
index 98efe9d5cb51fdfa996798e5f03e74fda5e9187b..3e586807a7795698b7ad3e46b87db674feb126db 100644 (file)
                 //only useful when one torrent
                 if (count == 1)
                 {
-                    name = [NSString stringWithUTF8String: info.name];
+                    name = @(info.name);
                     folder = info.isFolder;
                 }
             }
index 4730a7985eaaac66acd3b6f999760d8fc19af1bf..44025cc4a69e52164c342bda3bd42b8a3f24e1a8 100644 (file)
 
     for (tr_log_message * currentMessage = messages; currentMessage != NULL; currentMessage = currentMessage->next)
     {
-        NSString * name = currentMessage->name != NULL ? [NSString stringWithUTF8String: currentMessage->name]
+        NSString * name = currentMessage->name != NULL ? @(currentMessage->name)
                             : [[NSProcessInfo processInfo] processName];
 
-        NSString * file = [[[NSString stringWithUTF8String: currentMessage->file] lastPathComponent] stringByAppendingFormat: @":%d",
+        NSString * file = [[@(currentMessage->file) lastPathComponent] stringByAppendingFormat: @":%d",
                             currentMessage->line];
 
         NSDictionary * message  = [NSDictionary dictionaryWithObjectsAndKeys:
-                                    [NSString stringWithUTF8String: currentMessage->message], @"Message",
+                                    @(currentMessage->message), @"Message",
                                     [NSDate dateWithTimeIntervalSince1970: currentMessage->when], @"Date",
                                     @(currentIndex++), @"Index", //more accurate when sorting by date
                                     @(currentMessage->level), @"Level",
index 40c1c35acf2dc45eab00cb3ea240d0277f089c40..bd52cabaaa930b90268d1e311c29ffef55558c23 100644 (file)
@@ -38,7 +38,7 @@
 
 + (NSString *) ellipsis
 {
-    return [NSString stringWithUTF8String: "\xE2\x80\xA6"];
+    return @"\xE2\x80\xA6";
 }
 
 - (NSString *) stringByAppendingEllipsis
     if ((int)ratio == TR_RATIO_NA)
         return NSLocalizedString(@"N/A", "No Ratio");
     else if ((int)ratio == TR_RATIO_INF)
-        return [NSString stringWithUTF8String: "\xE2\x88\x9E"];
+        return @"\xE2\x88\x9E";
     else
     {
         if (ratio < 10.0)
index e5675017ca962b42f3078ba1ac0dcd78abcceabd..a5eafc3c354c16661dc7f8084e42e5e39dd7f9ed 100644 (file)
     [fDefaults setBool: encryptionMode == TR_ENCRYPTION_REQUIRED forKey: @"EncryptionRequire"];
 
     //download directory
-    NSString * downloadLocation = [[NSString stringWithUTF8String: tr_sessionGetDownloadDir(fHandle)] stringByStandardizingPath];
+    NSString * downloadLocation = [@(tr_sessionGetDownloadDir(fHandle)) stringByStandardizingPath];
     [fDefaults setObject: downloadLocation forKey: @"DownloadFolder"];
 
-    NSString * incompleteLocation = [[NSString stringWithUTF8String: tr_sessionGetIncompleteDir(fHandle)] stringByStandardizingPath];
+    NSString * incompleteLocation = [@(tr_sessionGetIncompleteDir(fHandle)) stringByStandardizingPath];
     [fDefaults setObject: incompleteLocation forKey: @"IncompleteDownloadFolder"];
 
     const BOOL useIncomplete = tr_sessionIsIncompleteDirEnabled(fHandle);
     const BOOL blocklist = tr_blocklistIsEnabled(fHandle);
     [fDefaults setBool: blocklist forKey: @"BlocklistNew"];
 
-    NSString * blocklistURL = [NSString stringWithUTF8String: tr_blocklistGetURL(fHandle)];
+    NSString * blocklistURL = @(tr_blocklistGetURL(fHandle));
     [fDefaults setObject: blocklistURL forKey: @"BlocklistURL"];
 
     //seed ratio
     const BOOL doneScriptEnabled = tr_sessionIsTorrentDoneScriptEnabled(fHandle);
     [fDefaults setBool: doneScriptEnabled forKey: @"DoneScriptEnabled"];
 
-    NSString * doneScriptPath = [NSString stringWithUTF8String: tr_sessionGetTorrentDoneScript(fHandle)];
+    NSString * doneScriptPath = @(tr_sessionGetTorrentDoneScript(fHandle));
     [fDefaults setObject: doneScriptPath forKey: @"DoneScriptPath"];
 
     //update gui if loaded
index 8d45b4dbb0fe2fda2c52fdfdf7b52c611c4c03bb..06965d4d697e119bd865db08c216b5c506d35c0b 100644 (file)
@@ -100,8 +100,8 @@ void metadataCallback(tr_torrent * torrent, void * torrentData)
 void renameCallback(tr_torrent * torrent, const char * oldPathCharString, const char * newNameCharString, int error, void * contextInfo)
 {
     @autoreleasepool {
-        NSString * oldPath = [NSString stringWithUTF8String: oldPathCharString];
-        NSString * newName = [NSString stringWithUTF8String: newNameCharString];
+        NSString * oldPath = @(oldPathCharString);
+        NSString * newName = @(newNameCharString);
 
         dispatch_async(dispatch_get_main_queue(), ^{
             NSDictionary * contextDict = [(NSDictionary *)contextInfo autorelease];
@@ -119,7 +119,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
     @autoreleasepool
     {
         NSError * localError;
-        if (![Torrent trashFile: [NSString stringWithUTF8String: filename] error: &localError])
+        if (![Torrent trashFile: @(filename) error: &localError])
         {
             tr_error_set_literal(error, [localError code], [[localError description] UTF8String]);
             return false;
@@ -283,7 +283,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
 
 - (NSString *) currentDirectory
 {
-    return [NSString stringWithUTF8String: tr_torrentGetCurrentDir(fHandle)];
+    return @(tr_torrentGetCurrentDir(fHandle));
 }
 
 - (void) getAvailability: (int8_t *) tab size: (NSInteger) size
@@ -410,7 +410,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
 
 - (NSString *) magnetLink
 {
-    return [NSString stringWithUTF8String: tr_torrentGetMagnetLink(fHandle)];
+    return @(tr_torrentGetMagnetLink(fHandle));
 }
 
 - (CGFloat) ratio
@@ -658,7 +658,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
 
 - (NSString *) name
 {
-    return fInfo->name != NULL ? [NSString stringWithUTF8String: fInfo->name] : fHashString;
+    return fInfo->name != NULL ? @(fInfo->name) : fHashString;
 }
 
 - (BOOL) isFolder
@@ -706,7 +706,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
     NSMutableArray * allTrackers = [NSMutableArray arrayWithCapacity: fInfo->trackerCount];
 
     for (NSInteger i=0; i < fInfo->trackerCount; i++)
-        [allTrackers addObject: [NSString stringWithUTF8String: fInfo->trackers[i].announce]];
+        [allTrackers addObject: @(fInfo->trackers[i].announce)];
 
     return allTrackers;
 }
@@ -742,7 +742,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
     NSUInteger newCount = 0;
     for (NSUInteger i = 0; i < fInfo->trackerCount; i++)
     {
-        if (![trackers containsObject: [NSString stringWithUTF8String: fInfo->trackers[i].announce]])
+        if (![trackers containsObject: @(fInfo->trackers[i].announce)])
             trackerStructs[newCount++] = fInfo->trackers[i];
     }
 
@@ -754,12 +754,12 @@ bool trashDataFile(const char * filename, tr_error ** error)
 
 - (NSString *) comment
 {
-    return fInfo->comment ? [NSString stringWithUTF8String: fInfo->comment] : @"";
+    return fInfo->comment ? @(fInfo->comment) : @"";
 }
 
 - (NSString *) creator
 {
-    return fInfo->creator ? [NSString stringWithUTF8String: fInfo->creator] : @"";
+    return fInfo->creator ? @(fInfo->creator) : @"";
 }
 
 - (NSDate *) dateCreated
@@ -790,7 +790,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
 
 - (NSString *) torrentLocation
 {
-    return fInfo->torrent ? [NSString stringWithUTF8String: fInfo->torrent] : @"";
+    return fInfo->torrent ? @(fInfo->torrent) : @"";
 }
 
 - (NSString *) dataLocation
@@ -813,7 +813,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
         if (location == NULL)
             return nil;
 
-        NSString * dataLocation = [NSString stringWithUTF8String: location];
+        NSString * dataLocation = @(location);
         free(location);
 
         return dataLocation;
@@ -838,7 +838,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
         if (location == NULL)
             return nil;
 
-        NSString * dataLocation = [NSString stringWithUTF8String: location];
+        NSString * dataLocation = @(location);
         free(location);
 
         return dataLocation;
@@ -946,7 +946,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
         return @"";
 
     NSString * error;
-    if (!(error = [NSString stringWithUTF8String: fStat->errorString])
+    if (!(error = @(fStat->errorString))
         && !(error = [NSString stringWithCString: fStat->errorString encoding: NSISOLatin1StringEncoding]))
         error = [NSString stringWithFormat: @"(%@)", NSLocalizedString(@"unreadable error", "Torrent -> error string unreadable")];
 
@@ -970,14 +970,14 @@ bool trashDataFile(const char * filename, tr_error ** error)
 
         [dict setObject: [self name] forKey: @"Name"];
         [dict setObject: @(peer->from) forKey: @"From"];
-        [dict setObject: [NSString stringWithUTF8String: peer->addr] forKey: @"IP"];
+        [dict setObject: @(peer->addr) forKey: @"IP"];
         [dict setObject: @(peer->port) forKey: @"Port"];
         [dict setObject: @(peer->progress) forKey: @"Progress"];
         [dict setObject: @(peer->isSeed) forKey: @"Seed"];
         [dict setObject: @(peer->isEncrypted) forKey: @"Encryption"];
         [dict setObject: @(peer->isUTP) forKey: @"uTP"];
-        [dict setObject: [NSString stringWithUTF8String: peer->client] forKey: @"Client"];
-        [dict setObject: [NSString stringWithUTF8String: peer->flagStr] forKey: @"Flags"];
+        [dict setObject: @(peer->client) forKey: @"Client"];
+        [dict setObject: @(peer->flagStr) forKey: @"Flags"];
 
         if (peer->isUploadingTo)
             [dict setObject: @(peer->rateToPeer_KBps) forKey: @"UL To Rate"];
@@ -1008,7 +1008,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
         NSMutableDictionary * dict = [NSMutableDictionary dictionaryWithCapacity: 3];
 
         [dict setObject: [self name] forKey: @"Name"];
-        [dict setObject: [NSString stringWithUTF8String: fInfo->webseeds[i]] forKey: @"Address"];
+        [dict setObject: @(fInfo->webseeds[i]) forKey: @"Address"];
 
         if (dlSpeeds[i] != -1.0)
             [dict setObject: @(dlSpeeds[i]) forKey: @"DL From Rate"];
@@ -1635,7 +1635,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
 
     for (int i=0; i < count; ++i)
     {
-        NSString * tracker = [NSString stringWithUTF8String: stats[i].host];
+        NSString * tracker = @(stats[i].host);
         if (!best || [tracker localizedCaseInsensitiveCompare: best] == NSOrderedAscending)
             best = tracker;
     }
@@ -1762,7 +1762,7 @@ bool trashDataFile(const char * filename, tr_error ** error)
         {
             tr_file * file = &fInfo->files[i];
 
-            NSString * fullPath = [NSString stringWithUTF8String: file->name];
+            NSString * fullPath = @(file->name);
             NSArray * pathComponents = [fullPath pathComponents];
 
             if (!tempNode)
index 6879a6d3f9f39942ed69ea2431a0beba7bfb0a02..3d45ff96e31f36e58c31ba29cbf62717bfe37d40 100644 (file)
 
 - (NSString *) host
 {
-    return [NSString stringWithUTF8String: fStat.host];
+    return @(fStat.host);
 }
 
 - (NSString *) fullAnnounceAddress
 {
-    return [NSString stringWithUTF8String: fStat.announce];
+    return @(fStat.announce);
 }
 
 - (NSInteger) tier
     {
         baseString = NSLocalizedString(@"Announce error", "Tracker last announce");
 
-        NSString * errorString = [NSString stringWithUTF8String: fStat.lastAnnounceResult];
+        NSString * errorString = @(fStat.lastAnnounceResult);
         if ([errorString isEqualToString: @""])
             baseString = [baseString stringByAppendingFormat: @": %@", dateString];
         else
     {
         baseString = NSLocalizedString(@"Scrape error", "Tracker last scrape");
 
-        NSString * errorString = [NSString stringWithUTF8String: fStat.lastScrapeResult];
+        NSString * errorString = @(fStat.lastScrapeResult);
         if ([errorString isEqualToString: @""])
             baseString = [baseString stringByAppendingFormat: @": %@", dateString];
         else