[Cdk-commits] [cdk/cdk] e15891: Upgraded to JavaCC 5.0

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

  Branch: refs/heads/master
  Home:   https://github.com/cdk/cdk
  Commit: e158914d2c71eca5fc640c54933aaddb1fa08ae6
      https://github.com/cdk/cdk/commit/e158914d2c71eca5fc640c54933aaddb1fa08ae6
  Author: Egon Willighagen <eg...@us...>
  Date:   2012-04-19 (Thu, 19 Apr 2012)

  Changed paths:
    M build.xml
    M jar/javacc.jar
    M jar/javacc.jar.meta

  Log Message:
  -----------
  Upgraded to JavaCC 5.0

Change-Id: I77b8fb2d3433f3bdb9691f98f5d8f188d6a91671

Signed-off-by: Rajarshi Guha <raj...@gm...>


  Commit: d6c7c32947df2fd38058e01740446f6bb82f35b5
      https://github.com/cdk/cdk/commit/d6c7c32947df2fd38058e01740446f6bb82f35b5
  Author: Egon Willighagen <eg...@us...>
  Date:   2012-04-19 (Thu, 19 Apr 2012)

  Changed paths:
    M src/main/org/openscience/cdk/DefaultChemObjectBuilder.java
    M src/main/org/openscience/cdk/debug/DebugChemObjectBuilder.java

  Log Message:
  -----------
  Patch to report candidate constructors when failing to instantiate a class because of an incorrect number of passed parameters. (fixes #2987186)

Example output:

No constructor found for NNAtom with the given number of parameters.
Candidates are: NNAtom(), NNAtom(String), NNAtom(IElement),
NNAtom(String, Point3d), NNAtom(String, Point2d)

Signed-off-by: Rajarshi Guha <raj...@gm...>

Conflicts:

	src/main/org/openscience/cdk/debug/DebugChemObjectBuilder.java
	src/main/org/openscience/cdk/nonotify/NoNotificationChemObjectBuilder.java

Change-Id: I7a31c6d819618df0f637655b9b149a80805c45ec


  Commit: 31b079b06cd3c46000da142ccb19263668823ada
      https://github.com/cdk/cdk/commit/31b079b06cd3c46000da142ccb19263668823ada
  Author: Egon Willighagen <eg...@us...>
  Date:   2012-04-19 (Thu, 19 Apr 2012)

  Changed paths:
    M src/test/org/openscience/cdk/AbstractChemObjectBuilderTest.java

  Log Message:
  -----------
  Added a unit test to see if we get suggestions

Signed-off-by: Rajarshi Guha <raj...@gm...>

Conflicts:

	src/test/org/openscience/cdk/AbstractChemObjectBuilderTest.java

Change-Id: Ia8fef98ea01adc6ecab9d83d1f46ea678167edf7


  Commit: 9711316d2044d0e97aed281005aacc4064132d8b
      https://github.com/cdk/cdk/commit/9711316d2044d0e97aed281005aacc4064132d8b
  Author: Egon Willighagen <eg...@us...>
  Date:   2012-04-19 (Thu, 19 Apr 2012)

  Changed paths:
    M src/main/org/openscience/cdk/silent/SilentChemObjectBuilder.java

  Log Message:
  -----------
  Added code to make suggestions to the silent builder too

Signed-off-by: Rajarshi Guha <raj...@gm...>

Conflicts:

	src/main/org/openscience/cdk/silent/SilentChemObjectBuilder.java

Change-Id: Ie860d797fa9c81cd6e867289e98745c5fd64b2a2


  Commit: b5d41fe3de1b3384aee39d4da3702789848e1f7b
      https://github.com/cdk/cdk/commit/b5d41fe3de1b3384aee39d4da3702789848e1f7b
  Author: Rajarshi Guha <raj...@gm...>
  Date:   2012-04-19 (Thu, 19 Apr 2012)

  Changed paths:
    M src/main/org/openscience/cdk/qsar/descriptors/molecular/TPSADescriptor.java
    M src/test/org/openscience/cdk/qsar/descriptors/molecular/TPSADescriptorTest.java

  Log Message:
  -----------
  Updated TPSA descriptor to properly cast to IRing rather than different Ring implementations. Added unit test

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

Conflicts:

	src/test/org/openscience/cdk/qsar/descriptors/molecular/TPSADescriptorTest.java


Compare: https://github.com/cdk/cdk/compare/d30874c...b5d41fe