[Cdk-commits] [cdk/cdk] e35e4e: Removed redundant code - we don't need to check wh...

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

  Branch: refs/heads/master
  Home:   https://github.com/cdk/cdk
  Commit: e35e4ef11763283f75ab447042a7085b728a99d8
      https://github.com/cdk/cdk/commit/e35e4ef11763283f75ab447042a7085b728a99d8
  Author: John May <joh...@gm...>
  Date:   2012-11-02 (Fri, 02 Nov 2012)

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

  Log Message:
  -----------
  Removed redundant code - we don't need to check whether the bond is already in the container as we create a new instance. We also don't need to check the array size as this is done by addBond(IBond)

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


  Commit: a2b5b1eb607af7129ef852dfc48ddc2c5d6768a8
      https://github.com/cdk/cdk/commit/a2b5b1eb607af7129ef852dfc48ddc2c5d6768a8
  Author: John May <joh...@gm...>
  Date:   2012-11-02 (Fri, 02 Nov 2012)

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

  Log Message:
  -----------
  Removed print to standard out from ChemObjectBuilders

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


  Commit: d2c3a5b343ef2166ce37a416596bc072397f8619
      https://github.com/cdk/cdk/commit/d2c3a5b343ef2166ce37a416596bc072397f8619
  Author: John May <joh...@gm...>
  Date:   2012-11-02 (Fri, 02 Nov 2012)

  Changed paths:
    M src/main/org/openscience/cdk/stereo/DoubleBondStereochemistry.java

  Log Message:
  -----------
  Added check to ensure a DoubleBondStereochemistry is never created with more then 2 bonds - this would cause errors with some methods.

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


  Commit: 04f6a1e6b9a0d58f7da177cc537362e9d5a0dc58
      https://github.com/cdk/cdk/commit/04f6a1e6b9a0d58f7da177cc537362e9d5a0dc58
  Author: John May <joh...@gm...>
  Date:   2012-11-02 (Fri, 02 Nov 2012)

  Changed paths:
    M src/main/org/openscience/cdk/AtomContainer.java
    M src/main/org/openscience/cdk/interfaces/IAtomContainer.java
    M src/main/org/openscience/cdk/silent/AtomContainer.java

  Log Message:
  -----------
  Added removal of stereo elements in 'removeAllElements()' - documentation has been updated

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


  Commit: 229700d26e913ed2ee77f325550b8b0b612cda73
      https://github.com/cdk/cdk/commit/229700d26e913ed2ee77f325550b8b0b612cda73
  Author: John May <joh...@gm...>
  Date:   2012-11-02 (Fri, 02 Nov 2012)

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

  Log Message:
  -----------
  Removed redundant code from ChemObj clone - the existing code did exactly what the copy constructor of HashMap does and thus provides a cleaner implementation

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

Conflicts:
	src/main/org/openscience/cdk/ChemObject.java
	src/main/org/openscience/cdk/silent/ChemObject.java


  Commit: c26154d78d12dced30f201d8263ae14f65379c93
      https://github.com/cdk/cdk/commit/c26154d78d12dced30f201d8263ae14f65379c93
  Author: John May <joh...@gm...>
  Date:   2012-11-02 (Fri, 02 Nov 2012)

  Changed paths:
    M src/META-INF/test-renderextra.cdkdepends

  Log Message:
  -----------
  Added cdk-silent dependency for test-renderextra

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


Compare: https://github.com/cdk/cdk/compare/a5ff5fd62bd6...c26154d78d12