[Cdk-commits] [cdk/cdk] 699995: The SMARTS '*' should match any atom, include hydr...

[GitHub <no...@gi...>]

  Branch: refs/heads/master
  Home:   https://github.com/cdk/cdk
  Commit: 6999951a9cc407c9f47764565d6e596acd2a73b7
      https://github.com/cdk/cdk/commit/6999951a9cc407c9f47764565d6e596acd2a73b7
  Author: John May <joh...@gm...>
  Date:   2014-07-16 (Wed, 16 Jul 2014)

  Changed paths:
    M tool/smarts/src/main/java/org/openscience/cdk/isomorphism/matchers/smarts/AnyAtom.java
    A tool/smarts/src/test/java/org/openscience/cdk/isomorphism/matchers/smarts/AnyAtomTest.java

  Log Message:
  -----------
  The SMARTS '*' should match any atom, include hydrogen. Hydrogen was not previously matched due to confussion over explicit-H matching in SMARTS.

Signed-off-by: Egon Willighagen <eg...@us...>


  Commit: 29d9a75899ac4e6c5351537070d724fd869a8004
      https://github.com/cdk/cdk/commit/29d9a75899ac4e6c5351537070d724fd869a8004
  Author: John May <joh...@gm...>
  Date:   2014-07-16 (Wed, 16 Jul 2014)

  Changed paths:
    M tool/smarts/src/test/java/org/openscience/cdk/smiles/smarts/parser/SMARTSSearchTest.java

  Log Message:
  -----------
  Resolve unit test regressions. Comment added to note the original bug report was wrong.

Signed-off-by: Egon Willighagen <eg...@us...>


Compare: https://github.com/cdk/cdk/compare/813637a728aa...29d9a75899ac