---

yaml
---
r: 63316
b: refs/heads/snap-stage3
c: 9f9e505
h: refs/heads/master
v: v3

Author: lilyball

[refs]

@@ -1,7 +1,7 @@
 ---
 refs/heads/master: 2d28d645422c1617be58c8ca7ad9a457264ca850
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
-refs/heads/snap-stage3: 821a962febc3261e97e31174dcb8cf142678bb4b
+refs/heads/snap-stage3: 9f9e50540581881a3c3a7304eaa2577afda59048
 refs/heads/try: 7b78b52e602bb3ea8174f9b2006bff3315f03ef9
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b

branches/snap-stage3/src/libextra/terminfo/parm.rs

@@ -68,7 +68,6 @@ pub fn expand(cap: &[u8], params: &mut [Param], sta: &mut [Param], dyn: &mut [Pa
 
     while i < cap.len() {
         cur = cap[i] as char;
-        debug!("current char: %c", cur);
         let mut old_state = state;
         match state {
             Nothing => {
@@ -134,43 +133,29 @@ pub fn expand(cap: &[u8], params: &mut [Param], sta: &mut [Param], dyn: &mut [Pa
                         (_, _) => return Err(~"non-numbers on stack with |")
                     },
                     'A' => match (stack.pop(), stack.pop()) {
-                        (Number(x), Number(y)) => {
-                            if x == 1 && y == 1 {
-                                stack.push(Number(1));
-                            } else {
-                                stack.push(Number(0));
-                            }
-                        },
-                        (_, _) => return Err(~"non-numbers on stack with logical and")
+                        (Number(0), Number(_)) => stack.push(Number(0)),
+                        (Number(_), Number(0)) => stack.push(Number(0)),
+                        (Number(_), Number(_)) => stack.push(Number(1)),
+                        _ => return Err(~"non-numbers on stack with logical and")
                     },
                     'O' => match (stack.pop(), stack.pop()) {
-                        (Number(x), Number(y)) => {
-                            if x == 1 && y == 1 {
-                                stack.push(Number(1));
-                            } else {
-                                stack.push(Number(0));
-                            }
-                        },
-                        (_, _) => return Err(~"non-numbers on stack with logical or")
+                        (Number(0), Number(0)) => stack.push(Number(0)),
+                        (Number(_), Number(_)) => stack.push(Number(1)),
+                        _ => return Err(~"non-numbers on stack with logical or")
                     },
                     '!' => match stack.pop() {
-                        Number(x) => {
-                            if x == 1 {
-                                stack.push(Number(0))
-                            } else {
-                                stack.push(Number(1))
-                            }
-                        },
+                        Number(0) => stack.push(Number(1)),
+                        Number(_) => stack.push(Number(0)),
                         _ => return Err(~"non-number on stack with logical not")
                     },
                     '~' => match stack.pop() {
                         Number(x) => stack.push(Number(!x)),
                         _         => return Err(~"non-number on stack with %~")
                     },
                     'i' => match (copy params[0], copy params[1]) {
-                        (Number(x), Number(y)) => {
-                            params[0] = Number(x + 1);
-                            params[1] = Number(y + 1);
+                        (Number(ref mut x), Number(ref mut y)) => {
+                            *x += 1;
+                            *y += 1;
                         },
                         (_, _) => return Err(~"first two params not numbers with %i")
                     },