Abstract methods not part of API are now protected, to discourage thi…

…er use Signed-off-by: Egon Willighagen <egonw@users.sourceforge.net>
cdk · Jun 23, 2010 · 7a92d42 · 7a92d42
1 parent 592b920
commit 7a92d42
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/.classpath b/.classpath
@@ -19,6 +19,6 @@
 	<classpathentry kind="lib" path="jar/cmlxom-2.5-b1.jar"/>
 	<classpathentry kind="lib" path="develjar/ojdcheck.jar"/>
 	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/>
-	<classpathentry kind="lib" path="jar/signatures-1.0-SNAPSHOT.jar"/>
+	<classpathentry kind="lib" path="jar/signatures-1.0.jar"/>
 	<classpathentry kind="output" path="bin"/>
 </classpath>
diff --git a/jar/signatures-1.0-SNAPSHOT.jar b/jar/signatures-1.0-SNAPSHOT.jar
diff --git a/jar/signatures-1.0.jar b/jar/signatures-1.0.jar
diff --git a/src/main/org/openscience/cdk/signature/AtomSignature.java b/src/main/org/openscience/cdk/signature/AtomSignature.java
@@ -127,14 +127,14 @@ public AtomSignature(int atomIndex, int height,
 
     @Override /** {@inheritDoc} */
     @TestMethod("getIntLabelTest")
-    public int getIntLabel(int vertexIndex) {
+    protected int getIntLabel(int vertexIndex) {
         IAtom atom = molecule.getAtom(vertexIndex);
         return atom.getMassNumber();
     }
 
     @Override /** {@inheritDoc} */
     @TestMethod("getConnectedTest")
-    public int[] getConnected(int vertexIndex) {
+    protected int[] getConnected(int vertexIndex) {
         IAtom atom  = this.molecule.getAtom(vertexIndex);
         List<IAtom> connected = this.molecule.getConnectedAtomsList(atom);
         int[] connectedIndices = new int[connected.size()];
@@ -147,7 +147,7 @@ public int[] getConnected(int vertexIndex) {
 
     @Override /** {@inheritDoc} */
     @TestMethod("getEdgeLabelTest")
-    public String getEdgeLabel(int vertexIndex, int otherVertexIndex) {
+    protected String getEdgeLabel(int vertexIndex, int otherVertexIndex) {
         IAtom atomA = this.molecule.getAtom(vertexIndex);
         IAtom atomB = this.molecule.getAtom(otherVertexIndex);
         IBond bond = this.molecule.getBond(atomA, atomB);
@@ -167,7 +167,7 @@ public String getEdgeLabel(int vertexIndex, int otherVertexIndex) {
 
     @Override /** {@inheritDoc} */
     @TestMethod("getVertexSymbolTest")
-    public String getVertexSymbol(int vertexIndex) {
+    protected String getVertexSymbol(int vertexIndex) {
         return this.molecule.getAtom(vertexIndex).getSymbol();
     }
 

diff --git a/src/main/org/openscience/cdk/signature/MoleculeSignature.java b/src/main/org/openscience/cdk/signature/MoleculeSignature.java
@@ -124,7 +124,7 @@ public MoleculeSignature(IMolecule molecule, int height) {
 
     @Override /** {@inheritDoc} */
     @TestMethod("getVertexCountTest")
-    public int getVertexCount() {
+    protected int getVertexCount() {
         return this.molecule.getAtomCount();
     }