Merge pull request #281 from divadretlaw/master

Removed the ++ and -- operators

Author: lattner

test/IDE/complete_stmt_controlling_expr.swift

@@ -378,7 +378,7 @@ func testCStyleForBodyI5(fooObject: FooStruct) {
 func testCStyleForBodyI6(fooObject: FooStruct) {
   var localInt = 42
   var localFooObject = FooStruct(localInt)
-  for var i = 0; ; unknown_var++ {
+  for var i = 0; ; unknown_var += 1 {
     #^C_STYLE_FOR_BODY_I_6^#
   }
 }

test/Interpreter/break_continue.swift

@@ -15,7 +15,7 @@ func test1() {
 func test2() {
   print("test2")
   var i : Int
-  for i=0;i<10;i += 1 {
+  for i in 0..<10 {
     if i > 2 {
       continue
     }
@@ -25,7 +25,7 @@ func test2() {
 func test3() {
   print("test3")
   var i : Int
-  for i=0;i<10;i += 1 {
+  for i in 0..<10 {
     if i > 2 {
       break
     }

test/Parse/recovery.swift

@@ -120,7 +120,7 @@ struct FixedSizedRefArrayOfOptional<T>
   init(capacity: Int)
   {
     buffer = Storage.Buffer(Storage.self, capacity, capacity)
-    for var i = 0; i < capacity; ++i {
+    for i in 0..<capacity {
       (buffer.baseAddress + i).initialize(.None)
     }
 
@@ -181,11 +181,12 @@ struct DictionaryIndex<Element> : BidirectionalIndexType {
   typealias Index = DictionaryIndex<Element>
 
   func predecessor() -> Index {
-    var j = self.offset
-    while --j > 0 {
+    var j = self.offset - 1
+    while j > 0 {
       if buffer[j] != nil {
         return Index(buffer: buffer, offset: j)
       }
+	  j -= 1
     }
     return self
   }
@@ -275,7 +276,7 @@ struct Dictionary<Key: Hashable, Value> : CollectionType, SequenceType {
       _buffer[i.offset] = Element(key: key, value: value)
 
       if !found {
-        ++_count
+        _count += 1
       }
     }
   }
@@ -360,7 +361,7 @@ struct Dictionary<Key: Hashable, Value> : CollectionType, SequenceType {
 
     // remove the element
     _buffer[pos.offset] = .None
-    --_count
+    _count -= 1
 
     // If we've put a hole in a chain of contiguous elements, some
     // element after the hole may belong where the new hole is.

test/Prototypes/MutableIndexableDict.swift

@@ -9,13 +9,15 @@ func foo() -> Int32 {
   }
 
   // CHECK: [[@LINE+1]]:9 -> [[@LINE+1]]:18 : (0 + 2)
-  while (--x > 0) {
-    if (x % 2 == 0) { continue }
+  while (x > 0) {
+    if (x % 2 == 0) { x -= 1; continue }
+	x -= 1
   }
 
   // CHECK: [[@LINE+1]]:9 -> [[@LINE+1]]:18 : ((0 + 4) - 5)
   while (x < 100) {
-    if (x++ == 10) { break }
+    if (x == 10) { break }
+	x += 1
   }
 
   // CHECK: [[@LINE+1]]:9 -> [[@LINE+1]]:18 : ((0 + 6) - 9)

test/SILGen/coverage_while.swift

@@ -5,7 +5,7 @@
 func ifexpr() -> Int {
   var x : Int = 0
   if true {
-    x++; 
+    x += 1;
   }
   return x
   // CHECK-LABEL: sil hidden  @_TF13sil_locations6ifexprFT_Si
@@ -18,9 +18,9 @@ func ifexpr() -> Int {
 func ifelseexpr() -> Int {
   var x : Int = 0
   if true {
-    x++; 
+    x += 1;
   } else {
-    x--;
+    x -= 1;
   }
   return x
   // CHECK-LABEL: sil hidden  @_TF13sil_locations10ifelseexprFT_Si
@@ -64,7 +64,7 @@ func ifexpr_rval() -> Int {
 
 // TODO: missing info on the first branch.
 func forstmt_empty_cond(i: Int) -> Int {
-  for var i=0;;++i {}
+  for var i=0;; i += 1 {}
     // CHECK-LABEL: sil hidden  @{{.*}}forstmt_empty_cond{{.*}}
     // CHECK: apply {{.*}} line:[[@LINE-2]]:13
     // CHECK: br [[TRUE_BB:bb[0-9]+]]
@@ -116,7 +116,7 @@ func multipleReturnsImplicitAndExplicit() {
   if x > 10 {
     return
   }
-  x++;
+  x += 1;
   // CHECK-LABEL: sil hidden  @_TF13sil_locations34multipleReturnsImplicitAndExplicitFT_T_
   // CHECK: cond_br
   // CHECK: br bb{{[0-9]+}} // {{.*}} line:[[@LINE-5]]:5:return
@@ -175,13 +175,13 @@ func testIf() {
 }
 
 func testFor() {
-  for (var i:Int = 0; i<10; i++) {
+  for i in 0..<10 {
     var y: Int = 300
-    y++;
+    y += 1;
     if true {
       break
     }
-    y--;
+    y -= 1;
     continue
   }
 
@@ -352,7 +352,7 @@ func testStringForEachStmt() {
 func testForStmt() {
   var i = 0
   var m = 0
-  for (i = 0; i < 10; ++i) {
+  for i in 0..<10 {
     m += 1
     if m == 15 {
       break

test/SILGen/sil_locations.swift

@@ -95,7 +95,7 @@ func let_decls() {
 
 
   let e = 42  // expected-note {{change 'let' to 'var' to make it mutable}} {{3-6=var}}
-  ++e         // expected-error {{cannot pass immutable value to mutating operator: 'e' is a 'let' constant}}
+  e += 1      // expected-error {{cannot pass immutable value to mutating operator: 'e' is a 'let' constant}}
 
   // <rdar://problem/16306600> QoI: passing a 'let' value as an inout results in an unfriendly diagnostic
   let f = 96 // expected-note {{change 'let' to 'var' to make it mutable}} {{3-6=var}}
@@ -322,8 +322,8 @@ func testSelectorStyleArguments1(x: Int, bar y: Int) {
 
 func testSelectorStyleArguments2(x: Int,
                                  bar y: Int) {
-  ++x  // expected-error {{cannot pass immutable value to mutating operator: 'x' is a 'let' constant}}
-  ++y  // expected-error {{cannot pass immutable value to mutating operator: 'y' is a 'let' constant}}
+  x += 1  // expected-error {{cannot pass immutable value to mutating operator: 'x' is a 'let' constant}}
+  y += 1  // expected-error {{cannot pass immutable value to mutating operator: 'y' is a 'let' constant}}
 }
 
 func invalid_inout(inout var x : Int) { // expected-error {{parameter may not have multiple 'inout', 'var', or 'let' specifiers}} {{26-30=}}

test/Sema/immutability.swift

@@ -73,7 +73,7 @@ var (sd2: Qtys)
 }
 
 for i in 0...5 {}
-for var i = 0, i2 = 1; i == 0; ++i {}
+for var i = 0, i2 = 1; i == 0; i += 1 {}
 while let v = o, z = o where v > z {}
 repeat {} while v == 0
 if let v = o, z = o where v > z {}

test/SourceKit/DocumentStructure/Inputs/main.swift

@@ -93,10 +93,10 @@ func test1(cp: ComputedProperty, sub: CC2) {
   var x = cp.value
   x = cp.readOnly
   cp.value = x
-  ++cp.value
+  cp.value += 1
   x = sub[0]
   sub[0] = x
-  ++sub[0]
+  sub[0] += 1
 }
 
 struct S2 {

test/SourceKit/Indexing/index.swift

@@ -243,7 +243,7 @@ tests.test("clSetKernelArgsListAPPLE") {
   // Validate our results
   //
   correct = 0
-  for(i = 0; i < count; i++)
+  for i in 0..<count
   {
     if(results[i] == data[i] * data[i]){
       correct += 1