Skip null bonds in max bond order calculation.

johnmay · johnmay · commit 78937238a54c · 2021-01-17T21:11:02.000Z
diff --git a/base/core/src/main/java/org/openscience/cdk/atomtype/CDKAtomTypeMatcher.java b/base/core/src/main/java/org/openscience/cdk/atomtype/CDKAtomTypeMatcher.java
@@ -494,7 +494,7 @@ private IAtomType perceiveCarbons(IAtomContainer atomContainer, IAtom atom,
     private Order getMaximumBondOrder(List<IBond> connectedBonds) {
     	IBond.Order max = IBond.Order.SINGLE;
     	for (IBond bond : connectedBonds) {
-            if (bond.getOrder().numeric() > max.numeric())
+            if (bond.getOrder() != null && bond.getOrder().numeric() > max.numeric())
             	max = bond.getOrder();
         }
         return max;

Original file line number	Diff line number	Diff line change
`@@ -494,7 +494,7 @@ private IAtomType perceiveCarbons(IAtomContainer atomContainer, IAtom atom,`
`494`	`494`	`private Order getMaximumBondOrder(List<IBond> connectedBonds) {`
`495`	`495`	`IBond.Order max = IBond.Order.SINGLE;`
`496`	`496`	`for (IBond bond : connectedBonds) {`
`497`		`- if (bond.getOrder().numeric() > max.numeric())`
	`497`	`+ if (bond.getOrder() != null && bond.getOrder().numeric() > max.numeric())`
`498`	`498`	`max = bond.getOrder();`
`499`	`499`	`}`
`500`	`500`	`return max;`