Remove extra paren

Author: lucic71

cpp2rust/converter/converter.cpp

@@ -2153,10 +2153,7 @@ bool Converter::VisitUnaryOperator(clang::UnaryOperator *expr) {
         expr->getType()->isIntegerType() && !expr->getType()->isBooleanType();
     PushParen paren_cast(*this, needs_int_cast);
     StrCat(token::kNot);
-    {
-      PushParen paren(*this);
-      ConvertCondition(sub_expr);
-    }
+    ConvertCondition(sub_expr);
     if (needs_int_cast) {
       ConvertCast(expr->getType());
     }

tests/unit/out/refcount/bool_condition_ptr.rs

@@ -16,13 +16,13 @@ fn main_0() -> i32 {
     if !(*p.borrow()).is_null() {
         assert!(true);
     }
-    if !(!(*p.borrow()).is_null()) {
+    if !!(*p.borrow()).is_null() {
         assert!(false);
     }
     if !(*np.borrow()).is_null() {
         assert!(false);
     }
-    if !(!(*np.borrow()).is_null()) {
+    if !!(*np.borrow()).is_null() {
         assert!(true);
     }
     let iter: Value<Ptr<i32>> = Rc::new(RefCell::new((*p.borrow()).clone()));
@@ -36,9 +36,9 @@ fn main_0() -> i32 {
     assert!(((*t3.borrow()) == 1));
     let t4: Value<i32> = Rc::new(RefCell::new(if !(*np.borrow()).is_null() { 1 } else { 0 }));
     assert!(((*t4.borrow()) == 0));
-    let t5: Value<i32> = Rc::new(RefCell::new((!(!(*p.borrow()).is_null()) as i32)));
+    let t5: Value<i32> = Rc::new(RefCell::new((!!(*p.borrow()).is_null() as i32)));
     assert!(((*t5.borrow()) == 0));
-    let t6: Value<i32> = Rc::new(RefCell::new((!(!(*np.borrow()).is_null()) as i32)));
+    let t6: Value<i32> = Rc::new(RefCell::new((!!(*np.borrow()).is_null() as i32)));
     assert!(((*t6.borrow()) == 1));
     let b2: Value<bool> = Rc::new(RefCell::new((!(*p.borrow()).is_null()).clone()));
     let b3: Value<bool> = Rc::new(RefCell::new((!(*np.borrow()).is_null()).clone()));

tests/unit/out/refcount/bool_condition_ptr_c.rs

@@ -16,13 +16,13 @@ fn main_0() -> i32 {
     if !(*p.borrow()).is_null() {
         assert!((1 != 0));
     }
-    if !(!(*p.borrow()).is_null()) {
+    if !!(*p.borrow()).is_null() {
         assert!((0 != 0));
     }
     if !(*np.borrow()).is_null() {
         assert!((0 != 0));
     }
-    if !(!(*np.borrow()).is_null()) {
+    if !!(*np.borrow()).is_null() {
         assert!((1 != 0));
     }
     let iter: Value<Ptr<i32>> = Rc::new(RefCell::new((*p.borrow()).clone()));
@@ -36,9 +36,9 @@ fn main_0() -> i32 {
     assert!(((*t3.borrow()) == 1));
     let t4: Value<i32> = Rc::new(RefCell::new(if !(*np.borrow()).is_null() { 1 } else { 0 }));
     assert!(((*t4.borrow()) == 0));
-    let t5: Value<i32> = Rc::new(RefCell::new((!(!(*p.borrow()).is_null()) as i32)));
+    let t5: Value<i32> = Rc::new(RefCell::new((!!(*p.borrow()).is_null() as i32)));
     assert!(((*t5.borrow()) == 0));
-    let t6: Value<i32> = Rc::new(RefCell::new((!(!(*np.borrow()).is_null()) as i32)));
+    let t6: Value<i32> = Rc::new(RefCell::new((!!(*np.borrow()).is_null() as i32)));
     assert!(((*t6.borrow()) == 1));
     let b2: Value<bool> = Rc::new(RefCell::new((!(*p.borrow()).is_null()).clone()));
     let b3: Value<bool> = Rc::new(RefCell::new((!(*np.borrow()).is_null()).clone()));

tests/unit/out/refcount/fn_ptr_as_condition.rs

@@ -43,7 +43,7 @@ fn main_0() -> i32 {
     });
     assert!(((*b.borrow()) == 5));
     let fn_: Value<FnPtr<fn(Ptr<i32>)>> = Rc::new(RefCell::new(FnPtr::null()));
-    if !(!(*fn_.borrow()).is_null()) {
+    if !!(*fn_.borrow()).is_null() {
         (*fn_.borrow_mut()) = FnPtr::<fn(Ptr<i32>)>::new(double_it_0);
     }
     let c: Value<i32> = Rc::new(RefCell::new(3));

tests/unit/out/refcount/vector.rs

@@ -17,7 +17,7 @@ fn main_0() -> i32 {
     assert!(((*v1.borrow()).len() as u64 == 0_u64));
     assert!((*v1.borrow()).is_empty());
     (*v1.borrow_mut()).push(1);
-    assert!(!((*v1.borrow()).is_empty()));
+    assert!(!(*v1.borrow()).is_empty());
     (*v1.borrow_mut()).pop();
     assert!((*v1.borrow()).is_empty());
     let s1: Value<u64> = Rc::new(RefCell::new((*v1.borrow()).len() as u64));

tests/unit/out/unsafe/bool_condition_enum.rs

@@ -46,6 +46,6 @@ unsafe fn main_0() -> i32 {
     let mut t9: i32 = (!(code != Code::from(0)) as i32);
     assert!(((t9) == (1)));
     let mut b4: bool = (code != Code::from(0));
-    assert!(!(b4));
+    assert!(!b4);
     return 0;
 }

tests/unit/out/unsafe/bool_condition_enum_c.rs

@@ -46,6 +46,6 @@ unsafe fn main_0() -> i32 {
     let mut t9: i32 = (!(code != Code::from(0)) as i32);
     assert!(((t9) == (1)));
     let mut b4: bool = (code != Code::from(0));
-    assert!(((!(b4) as i32) != 0));
+    assert!(((!b4 as i32) != 0));
     return 0;
 }

tests/unit/out/unsafe/bool_condition_ptr.rs

@@ -18,13 +18,13 @@ unsafe fn main_0() -> i32 {
     if !(p).is_null() {
         assert!(true);
     }
-    if !(!(p).is_null()) {
+    if !!(p).is_null() {
         assert!(false);
     }
     if !(np).is_null() {
         assert!(false);
     }
-    if !(!(np).is_null()) {
+    if !!(np).is_null() {
         assert!(true);
     }
     let mut iter: *mut i32 = p;
@@ -38,13 +38,13 @@ unsafe fn main_0() -> i32 {
     assert!(((t3) == (1)));
     let mut t4: i32 = if !(np).is_null() { 1 } else { 0 };
     assert!(((t4) == (0)));
-    let mut t5: i32 = (!(!(p).is_null()) as i32);
+    let mut t5: i32 = (!!(p).is_null() as i32);
     assert!(((t5) == (0)));
-    let mut t6: i32 = (!(!(np).is_null()) as i32);
+    let mut t6: i32 = (!!(np).is_null() as i32);
     assert!(((t6) == (1)));
     let mut b2: bool = !(p).is_null();
     let mut b3: bool = !(np).is_null();
     assert!(b2);
-    assert!(!(b3));
+    assert!(!b3);
     return 0;
 }

tests/unit/out/unsafe/bool_condition_ptr_c.rs

@@ -18,13 +18,13 @@ unsafe fn main_0() -> i32 {
     if !(p).is_null() {
         assert!((1 != 0));
     }
-    if !(!(p).is_null()) {
+    if !!(p).is_null() {
         assert!((0 != 0));
     }
     if !(np).is_null() {
         assert!((0 != 0));
     }
-    if !(!(np).is_null()) {
+    if !!(np).is_null() {
         assert!((1 != 0));
     }
     let mut iter: *mut i32 = p;
@@ -38,13 +38,13 @@ unsafe fn main_0() -> i32 {
     assert!(((t3) == (1)));
     let mut t4: i32 = if !(np).is_null() { 1 } else { 0 };
     assert!(((t4) == (0)));
-    let mut t5: i32 = (!(!(p).is_null()) as i32);
+    let mut t5: i32 = (!!(p).is_null() as i32);
     assert!(((t5) == (0)));
-    let mut t6: i32 = (!(!(np).is_null()) as i32);
+    let mut t6: i32 = (!!(np).is_null() as i32);
     assert!(((t6) == (1)));
     let mut b2: bool = !(p).is_null();
     let mut b3: bool = !(np).is_null();
     assert!(b2);
-    assert!(((!(b3) as i32) != 0));
+    assert!(((!b3 as i32) != 0));
     return 0;
 }

tests/unit/out/unsafe/fn_ptr_as_condition.rs

@@ -38,7 +38,7 @@ unsafe fn main_0() -> i32 {
     });
     assert!(((b) == (5)));
     let mut fn_: Option<unsafe fn(*mut i32)> = None;
-    if !(!(fn_).is_none()) {
+    if !!(fn_).is_none() {
         fn_ = Some(double_it_0);
     }
     let mut c: i32 = 3;