Use prefix increment in for loops.
authorHaojian Wu <hokein.wu@gmail.com>
Thu, 30 Mar 2017 20:56:21 +0000 (22:56 +0200)
committerHaojian Wu <hokein.wu@gmail.com>
Thu, 30 Mar 2017 20:56:21 +0000 (22:56 +0200)
atom/browser/browser_mac.mm
atom/browser/ui/cocoa/atom_touch_bar.mm

index f231d7a..1a15bc0 100644 (file)
@@ -64,7 +64,7 @@ bool Browser::RemoveAsDefaultProtocolClient(const std::string& protocol,
   // On macOS, we can't query the default, but the handlers list seems to put
   // Apple's defaults first, so we'll use the first option that isn't our bundle
   CFStringRef other = nil;
-  for (CFIndex i = 0; i < CFArrayGetCount(bundleList); i++) {
+  for (CFIndex i = 0; i < CFArrayGetCount(bundleList); ++i) {
     other = base::mac::CFCast<CFStringRef>(CFArrayGetValueAtIndex(bundleList,
                                                                   i));
     if (![identifier isEqualToString: (__bridge NSString *)other]) {
index 6e3fc83..6992668 100644 (file)
@@ -414,7 +414,7 @@ static NSString* const ImageScrubberItemIdentifier = @"scrubber.image.item";
 
   NSMutableArray* generatedItems = [NSMutableArray array];
   NSMutableArray* identifiers = [self identifiersFromSettings:items];
-  for (NSUInteger i = 0; i < [identifiers count]; i++) {
+  for (NSUInteger i = 0; i < [identifiers count]; ++i) {
     if ([identifiers objectAtIndex:i] != NSTouchBarItemIdentifierOtherItemsProxy) {
       NSTouchBarItem* generatedItem = [self makeItemForIdentifier:[identifiers objectAtIndex:i]];
       if (generatedItem) {
@@ -474,7 +474,7 @@ static NSString* const ImageScrubberItemIdentifier = @"scrubber.image.item";
   settings.Get("segments", &segments);
 
   control.segmentCount = segments.size();
-  for (size_t i = 0; i < segments.size(); i++) {
+  for (size_t i = 0; i < segments.size(); ++i) {
     std::string label;
     gfx::Image image;
     bool enabled = true;