Fix more warnings (bool cast)

zhi-wang · zhi-wang · commit 419d4106bec2 · 2022-06-16T05:46:22.000-05:00
diff --git a/src/bonded/eangle.cpp b/src/bonded/eangle.cpp
@@ -15,7 +15,7 @@ void eangleData(RcOp op)
       not use(Potent::ANGTOR) and not use(Potent::CHGFLX))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(iang, ak, anat, afld, angtyp);
@@ -81,10 +81,10 @@ void eangleData(RcOp op)
 TINKER_FVOID1(acc1, cu0, eangle, int);
 void eangle(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_ea, virial_ea);
diff --git a/src/bonded/eangtor.cpp b/src/bonded/eangtor.cpp
@@ -12,7 +12,7 @@ void eangtorData(RcOp op)
    if (not use(Potent::ANGTOR))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       nangtor = 0;
@@ -54,10 +54,10 @@ void eangtorData(RcOp op)
 TINKER_FVOID1(acc1, cu0, eangtor, int);
 void eangtor(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_eat, virial_eat);
diff --git a/src/bonded/ebond.cpp b/src/bonded/ebond.cpp
@@ -14,7 +14,7 @@ void ebondData(RcOp op)
       not use(Potent::CHGFLX))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(ibnd, bl, bk);
@@ -66,10 +66,10 @@ void ebondData(RcOp op)
 TINKER_FVOID1(acc1, cu0, ebond, int);
 void ebond(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_eb, virial_eb);
diff --git a/src/bonded/egeom.cpp b/src/bonded/egeom.cpp
@@ -12,7 +12,7 @@ void egeomData(RcOp op)
    if (not use(Potent::GEOM))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       npfix = 0;
@@ -94,10 +94,10 @@ void egeomData(RcOp op)
 TINKER_FVOID1(acc1, cu0, egeom, int);
 void egeom(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_eg, virial_eg);
diff --git a/src/bonded/eimprop.cpp b/src/bonded/eimprop.cpp
@@ -12,7 +12,7 @@ void eimpropData(RcOp op)
    if (not use(Potent::IMPROP))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {}
 
@@ -44,10 +44,10 @@ void eimpropData(RcOp op)
 TINKER_FVOID1(acc1, cu0, eimprop, int);
 void eimprop(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_eid, virial_eid);
diff --git a/src/bonded/eimptor.cpp b/src/bonded/eimptor.cpp
@@ -12,7 +12,7 @@ void eimptorData(RcOp op)
    if (not use(Potent::IMPTORS))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(iitors, itors1, itors2, itors3);
@@ -54,10 +54,10 @@ void eimptorData(RcOp op)
 TINKER_FVOID1(acc1, cu0, eimptor, int);
 void eimptor(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_eit, virial_eit);
diff --git a/src/bonded/eopbend.cpp b/src/bonded/eopbend.cpp
@@ -14,7 +14,7 @@ void eopbendData(RcOp op)
    if (not use(Potent::OPBEND))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(iopb, opbk);
@@ -68,10 +68,10 @@ void eopbendData(RcOp op)
 TINKER_FVOID1(acc1, cu0, eopbend, int);
 void eopbend(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_eopb, virial_eopb);
diff --git a/src/bonded/epitors.cpp b/src/bonded/epitors.cpp
@@ -12,7 +12,7 @@ void epitorsData(RcOp op)
    if (not use(Potent::PITORS))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(ipit, kpit);
@@ -55,10 +55,10 @@ void epitorsData(RcOp op)
 TINKER_FVOID1(acc1, cu0, epitors, int);
 void epitors(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_ept, virial_ept);
diff --git a/src/bonded/estrbnd.cpp b/src/bonded/estrbnd.cpp
@@ -12,7 +12,7 @@ void estrbndData(RcOp op)
    if (not use(Potent::STRBND))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(isb, sbk);
@@ -57,10 +57,10 @@ void estrbndData(RcOp op)
 TINKER_FVOID1(acc1, cu0, estrbnd, int);
 void estrbnd(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_eba, virial_eba);
diff --git a/src/bonded/estrtor.cpp b/src/bonded/estrtor.cpp
@@ -12,7 +12,7 @@ void estrtorData(RcOp op)
    if (not use(Potent::STRTOR))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       nstrtor = 0;
@@ -54,10 +54,10 @@ void estrtorData(RcOp op)
 TINKER_FVOID1(acc1, cu0, estrtor, int);
 void estrtor(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_ebt, virial_ebt);
diff --git a/src/bonded/etors.cpp b/src/bonded/etors.cpp
@@ -12,7 +12,7 @@ void etorsData(RcOp op)
    if (not use(Potent::TORSION) and not use(Potent::STRTOR) and not use(Potent::ANGTOR))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(itors, tors1, tors2, tors3, tors4, tors5, tors6);
@@ -59,10 +59,10 @@ void etorsData(RcOp op)
 TINKER_FVOID1(acc1, cu0, etors, int);
 void etors(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_et, virial_et);
diff --git a/src/bonded/etortor.cpp b/src/bonded/etortor.cpp
@@ -16,7 +16,7 @@ void etortorData(RcOp op)
    if (not use(Potent::TORTOR))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(ibitor, itt, tnx, tny, ttx, tty, tbf, tbx, tby, tbxy, chkttor_ia_);
@@ -139,10 +139,10 @@ void etortorData(RcOp op)
 TINKER_FVOID1(acc1, cu0, etortor, int);
 void etortor(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_ett, virial_ett);
diff --git a/src/bonded/eurey.cpp b/src/bonded/eurey.cpp
@@ -12,7 +12,7 @@ void eureyData(RcOp op)
    if (not use(Potent::UREY))
       return;
 
-   bool rc_a = rc_flag & calc::analyz;
+   auto rc_a = rc_flag & calc::analyz;
 
    if (op & RcOp::DEALLOC) {
       darray::deallocate(iury, uk, ul);
@@ -60,10 +60,10 @@ void eureyData(RcOp op)
 TINKER_FVOID1(acc1, cu0, eurey, int);
 void eurey(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    if (rc_a) {
       zeroOnHost(energy_eub, virial_eub);
diff --git a/src/cu/evalence.cu b/src/cu/evalence.cu
@@ -817,10 +817,10 @@ void evalence_cu2(int vers, bool flag_bond, bool flag_angle, bool flag_strbnd, b
 
 void evalence_cu(int vers)
 {
-   bool rc_a = rc_flag & calc::analyz;
-   bool do_e = vers & calc::energy;
-   bool do_v = vers & calc::virial;
-   bool do_g = vers & calc::grad;
+   auto rc_a = rc_flag & calc::analyz;
+   auto do_e = vers & calc::energy;
+   auto do_v = vers & calc::virial;
+   auto do_g = vers & calc::grad;
 
    bool flag_bond = use(Potent::BOND);
    bool flag_angle = use(Potent::ANGLE);
diff --git a/src/cu/mdintg.cu b/src/cu/mdintg.cu
@@ -125,12 +125,12 @@ void mdrest_cu(int istep)
    auto totmass = molcul::totmass;
    vel_prec vtot1 = 0, vtot2 = 0, vtot3 = 0;
 
-   bool copyout = inform::debug or not bound::use_bounds;
+   bool copyout = static_cast<bool>(inform::debug) or not static_cast<bool>(bound::use_bounds);
    mdrestRemovePbcMomentum_cu(copyout, vtot1, vtot2, vtot3);
 
    // print the translational velocity of the overall system
 
-   mdrestPrintP1(inform::debug, vtot1, vtot2, vtot3, totmass);
+   mdrestPrintP1(static_cast<bool>(inform::debug), vtot1, vtot2, vtot3, totmass);
 
    if (not bound::use_bounds) {
       throwExceptionMissingFunction("mdrestRemoveAngularMomentum_cu", __FILE__, __LINE__);
