Browse Source

Use prefix increment in for loops.

Haojian Wu 8 years ago
parent
commit
ff745e1a82
2 changed files with 3 additions and 3 deletions
  1. 1 1
      atom/browser/browser_mac.mm
  2. 2 2
      atom/browser/ui/cocoa/atom_touch_bar.mm

+ 1 - 1
atom/browser/browser_mac.mm

@@ -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]) {

+ 2 - 2
atom/browser/ui/cocoa/atom_touch_bar.mm

@@ -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;