fix warning on cran checks

DESCRIPTION

@@ -1,6 +1,6 @@
 Package: dodgr
 Title: Distances on Directed Graphs
-Version: 0.2.12.047
+Version: 0.2.12.048
 Authors@R: c(
     person("Mark", "Padgham", , "[email protected]", role = c("aut", "cre")),
     person("Andreas", "Petutschnig", role = "aut"),

NEWS.md

@@ -16,6 +16,7 @@ Minor changes:
 - Slightly extend categorical routing vignette
 - fix caching bug in dodgr_centrality (#138), thanks to @mem48
 - Fix bug with bbox in `dodgr_streetnet`, thanks to @LeshunXu (#174)
+- Fix bug in src/graph-contract.cpp that led to CRAN warning (`|` -> `||`)
 
 # v 0.2.12
 

codemeta.json

@@ -7,7 +7,7 @@
   "codeRepository": "https://github.com/ATFutures/dodgr",
   "issueTracker": "https://github.com/ATFutures/dodgr/issues",
   "license": "https://spdx.org/licenses/GPL-3.0",
-  "version": "0.2.12.047",
+  "version": "0.2.12.048",
   "programmingLanguage": {
     "@type": "ComputerLanguage",
     "name": "R",

cran-comments.md

@@ -1,9 +1,8 @@
 # CRAN notes for dodgr_0.2.12 submission
 
-This submission rectifies the errors generated by the previous submission on
-some systems. They were due to inadvertent inclusion of an OS-specific C++
-header file. That header has now been removed. The single https redirect has
-also been rectified.
+This submission rectifies the warnings and errors generated by the previous submission on some systems, including:
+1. Replacing the bitwise with boolean operator in src code
+
 
 ## Notes
 

src/graph-contract.cpp

@@ -212,7 +212,7 @@ void graph_contract::contract_graph (vertex_map_t &vertex_map,
                     // It is possible to have repeated values of two_nbs;
                     // calling these a second time should then just lead to no
                     // contraction
-                    if ((vt_from == "") | (vt_to == ""))
+                    if ((vt_from == "") || (vt_to == ""))
                         continue;
 
                     graph_contract::contract_one_edge (vert2edge_map,