Skip to content

Commit

Permalink
The AtomType(String) constructor is updated so that only formal charg…
Browse files Browse the repository at this point in the history
…e is set to 0 as indicated in the Javadocs. All other fields are set ot UNSET. Javadocs were updated to make this explicit
  • Loading branch information
rajarshi committed Nov 29, 2009
1 parent 2cb55bd commit 8206e95
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/main/org/openscience/cdk/AtomType.java
Original file line number Diff line number Diff line change
Expand Up @@ -102,18 +102,18 @@ public class AtomType extends Isotope implements IAtomType, Serializable, Clonea
/**
* String representing the identifier for this atom type with null as default.
*/
private String identifier;
private String identifier = (String) CDKConstants.UNSET;

/**
* Constructor for the AtomType object. Defaults to a zero formal charge.
* Constructor for the AtomType object.
*
* Defaults to a zero formal charge. All
* other fields are set to {@link org.openscience.cdk.CDKConstants.UNSET}.
*
* @param elementSymbol Symbol of the atom
*/
public AtomType(String elementSymbol) {
super(elementSymbol);
this.identifier = null;
this.formalNeighbourCount = 0;
this.electronValency = 0;
this.formalCharge = 0;
}

Expand Down

0 comments on commit 8206e95

Please sign in to comment.