From: <raj...@us...> - 2008-09-08 05:11:21
|
Revision: 12227 http://cdk.svn.sourceforge.net/cdk/?rev=12227&view=rev Author: rajarshi Date: 2008-09-08 05:11:19 +0000 (Mon, 08 Sep 2008) Log Message: ----------- Fixed the return value in getIsomorphAtomsMap. Fixes bug #2028685 Modified Paths: -------------- cdk/branches/cdk-1.2.x/src/main/org/openscience/cdk/isomorphism/UniversalIsomorphismTester.java Modified: cdk/branches/cdk-1.2.x/src/main/org/openscience/cdk/isomorphism/UniversalIsomorphismTester.java =================================================================== --- cdk/branches/cdk-1.2.x/src/main/org/openscience/cdk/isomorphism/UniversalIsomorphismTester.java 2008-09-08 05:07:38 UTC (rev 12226) +++ cdk/branches/cdk-1.2.x/src/main/org/openscience/cdk/isomorphism/UniversalIsomorphismTester.java 2008-09-08 05:11:19 UTC (rev 12227) @@ -178,6 +178,8 @@ * @param g2 second molecule. May be an IQueryAtomContainer. * @return the first isomorph atom mapping found projected on g1. * This is a List of RMap objects containing Ids of matching atoms. + * @throws org.openscience.cdk.exception.CDKException if the first molecules is not an instance of + * {@link org.openscience.cdk.isomorphism.matchers.IQueryAtomContainer} */ public static List<RMap> getIsomorphAtomsMap(IAtomContainer g1, IAtomContainer g2) throws CDKException { if (g1 instanceof IQueryAtomContainer) @@ -191,8 +193,7 @@ } else if (list.isEmpty()) { return null; } else { - // FIXME: is this correct?? - return (List)list.get(0); + return list; } } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |