Skip to content

Commit

Permalink
Redundant semi-colon
Browse files Browse the repository at this point in the history
  • Loading branch information
johnmay committed Feb 10, 2022
1 parent 7cc1f17 commit 1ca0b58
Show file tree
Hide file tree
Showing 53 changed files with 66 additions and 69 deletions.
Expand Up @@ -180,7 +180,7 @@ public Double getNaturalAbundance() {
@Override
public Double getExactMass() {
return exactMass;
};
}

@Override
public Integer getMassNumber() {
Expand Down
Expand Up @@ -256,7 +256,7 @@ public Double getNaturalAbundance() {
@Override
public Double getExactMass() {
return exactMass;
};
}

@Override
public Integer getMassNumber() {
Expand Down
Expand Up @@ -51,5 +51,5 @@ public enum SgroupKey {
DataFieldFormat,
DataFieldUnits,
DataDisplayInformation,
Data;
Data
}
Expand Up @@ -28,4 +28,4 @@ public enum ReactionRole {
Agent,
Product,
None
};
}
Expand Up @@ -47,7 +47,7 @@ public void setOperator(String str) {}
public boolean matches(IAtom atom) {
return this.getAtomicNumber().equals(atom.getAtomicNumber()) && Objects.equals(this.getFormalCharge(), atom.getFormalCharge())
&& this.getID().equals(atom.getID());
};
}

@Override
public String toString() {
Expand Down
Expand Up @@ -60,21 +60,21 @@ public ReactionSpecification(String specificationReference, String implementatio
@Override
public String getSpecificationReference() {
return this.specificationReference;
};
}

@Override
public String getImplementationTitle() {
return this.implementationTitle;
};
}

@Override
public String getImplementationIdentifier() {
return this.implementationIdentifier;
};
}

@Override
public String getImplementationVendor() {
return this.implementationVendor;
};
}

}
Expand Up @@ -155,7 +155,7 @@ public void testInitiate_IAtomContainerSet_IAtomContainerSet() throws Exception
expected1.addBond(0, 3, IBond.Order.SINGLE);
expected1.addBond(0, 4, IBond.Order.SINGLE);
expected1.addBond(1, 5, IBond.Order.SINGLE);
expected1.addBond(1, 6, IBond.Order.SINGLE);;
expected1.addBond(1, 6, IBond.Order.SINGLE);
AtomContainerManipulator.percieveAtomTypesAndConfigureAtoms(expected1);
IAtomContainer product1 = setOfReactions.getReaction(0).getProducts().getAtomContainer(0);
IQueryAtomContainer queryAtom = QueryAtomContainerCreator.createSymbolAndChargeQueryContainer(expected1);
Expand Down
Expand Up @@ -155,7 +155,7 @@ public void testInitiate_IAtomContainerSet_IAtomContainerSet() throws Exception
expected1.addBond(0, 3, IBond.Order.SINGLE);
expected1.addBond(0, 4, IBond.Order.SINGLE);
expected1.addBond(1, 5, IBond.Order.SINGLE);
expected1.addBond(1, 6, IBond.Order.SINGLE);;
expected1.addBond(1, 6, IBond.Order.SINGLE);
AtomContainerManipulator.percieveAtomTypesAndConfigureAtoms(expected1);
IAtomContainer product1 = setOfReactions.getReaction(0).getProducts().getAtomContainer(0);
IQueryAtomContainer queryAtom = QueryAtomContainerCreator.createSymbolAndChargeQueryContainer(expected1);
Expand Down
Expand Up @@ -49,7 +49,7 @@ abstract public class AbstractSybylAtomTypeTest extends AbstractAtomTypeTest {
@Override
public String getAtomTypeListName() {
return ATOMTYPE_LIST;
};
}

@Override
public AtomTypeFactory getFactory() {
Expand Down
Expand Up @@ -43,7 +43,7 @@ abstract public class AbstractCDKAtomTypeTest extends AbstractAtomTypeTest {
@Override
public String getAtomTypeListName() {
return ATOMTYPE_LIST;
};
}

@Override
public AtomTypeFactory getFactory() {
Expand Down
Expand Up @@ -235,7 +235,7 @@ public void testNoAtomsIsConnected() {
public void copySgroups() throws Exception {
String filename = "sgroup-split.mol";
try (InputStream ins = this.getClass().getResourceAsStream(filename);
ISimpleChemObjectReader reader = new MDLV2000Reader(ins);
ISimpleChemObjectReader reader = new MDLV2000Reader(ins)
) {
ChemFile content = (ChemFile) reader.read((ChemObject) new ChemFile());
List<IAtomContainer> cList = ChemFileManipulator.getAllAtomContainers(content);
Expand Down
Expand Up @@ -19,7 +19,7 @@ public abstract class AbstractFingerprinterTest extends CDKTestCase {

public IFingerprinter getBitFingerprinter() {
throw new IllegalAccessError("This method should be overwritten " + "by subclasses unit tests");
};
}

/* override if method is implemented */
@Test(expected = UnsupportedOperationException.class)
Expand Down
Expand Up @@ -343,7 +343,7 @@ class DummyChemObjectListener implements IChemObjectListener {

@Test
@Override
public void stateChanged(IChemObjectChangeEvent event) {};
public void stateChanged(IChemObjectChangeEvent event) {}
}

@Test
Expand Down
Expand Up @@ -218,8 +218,6 @@ public DescriptorSpecification getSpecification() {
this.getClass().getName(), "The Chemistry Development Kit");
}

;

/**
* Sets the parameters attribute of the WHIMDescriptor object.
*
Expand Down
Expand Up @@ -507,6 +507,6 @@ public <T extends IGeneratorParameter<?>> boolean hasParameter(Class<T> param) {
*/
public void setNotification(boolean notification) {
this.notification = notification;
};
}

}
Expand Up @@ -124,7 +124,7 @@ public class CPKAtomColors implements IAtomColorer, java.io.Serializable {
ATOM_COLORS_MASSNUM.put(53, ATOM_COLORS_SYMBOL.get("I"));
ATOM_COLORS_MASSNUM.put(56, ATOM_COLORS_SYMBOL.get("BA"));
ATOM_COLORS_MASSNUM.put(79, ATOM_COLORS_SYMBOL.get("AU"));
};
}

//////////
// METHODS
Expand Down
Expand Up @@ -46,4 +46,4 @@ public Boolean getValue() {
public void setValue(Boolean value) {
this.value = value;
}
};
}
Expand Up @@ -39,7 +39,7 @@ public class TextGroupElement extends TextElement {
*/
public enum Position {
NW, SW, SE, NE, S, N, W, E
};
}

/**
* A string of text that should be shown around the parent.
Expand Down
Expand Up @@ -52,7 +52,7 @@ public enum TYPE {
* point in the line.
*/
public enum Direction {
toFirst, toSecond;
toFirst, toSecond
}

/**
Expand Down
Expand Up @@ -31,5 +31,5 @@
* @cdk.githash
*/
public enum Type {
MoveTo, LineTo, QuadTo, CubicTo, Close;
MoveTo, LineTo, QuadTo, CubicTo, Close
}
Expand Up @@ -167,7 +167,7 @@ public Boolean getDefault() {
*/
public enum Shape {
OVAL, SQUARE
};
}

/**
* Shape to be used when drawing atoms in compact mode,
Expand Down
Expand Up @@ -98,7 +98,7 @@ public static enum CoordinateCoverage {
*/
NONE

};
}

/**
* Adds an automatically calculated offset to the coordinates of all atoms
Expand Down
Expand Up @@ -36,7 +36,7 @@
@Deprecated
public class INChIContentProcessorTool {

private static final ILoggingTool logger = LoggingToolFactory.createLoggingTool(INChIContentProcessorTool.class); ;
private static final ILoggingTool logger = LoggingToolFactory.createLoggingTool(INChIContentProcessorTool.class);

public INChIContentProcessorTool() {}

Expand Down
Expand Up @@ -49,7 +49,7 @@ public boolean matches(IBond bond) {
return true;
} // else
return false;
};
}

@Override
public void setAtoms(IAtom[] atoms) {
Expand Down
Expand Up @@ -50,7 +50,7 @@ public boolean matches(IBond bond) {
return true;
} // else
return false;
};
}

@Override
public void setAtoms(IAtom[] atoms) {
Expand Down
Expand Up @@ -46,7 +46,7 @@ public boolean matches(IAtom atom) {
int requiredCharge = this.getFormalCharge() == null ? 0 : this.getFormalCharge();
int foundCharge = atom.getFormalCharge() == null ? 0 : atom.getFormalCharge();
return this.getSymbol().equals(atom.getSymbol()) && requiredCharge == foundCharge;
};
}

@Override
public String toString() {
Expand Down
Expand Up @@ -52,7 +52,7 @@ else if (ID == null && HCount != 0) {
return (this.getImplicitHydrogenCount() == HCount);
} else
return this.getSymbol().equals(atom.getSymbol());
};
}

public void setOperator(String str) {
ID = str;
Expand Down
Expand Up @@ -301,11 +301,11 @@ public void xtestPyrrole() throws Exception {
Assert.assertTrue(dbst.isOK(enol));

// now check whether it did the right thing
Assert.assertEquals(Order.DOUBLE, enol.getBond(0).getOrder());;
Assert.assertEquals(Order.SINGLE, enol.getBond(1).getOrder());;
Assert.assertEquals(Order.DOUBLE, enol.getBond(2).getOrder());;
Assert.assertEquals(Order.SINGLE, enol.getBond(3).getOrder());;
Assert.assertEquals(Order.SINGLE, enol.getBond(4).getOrder());;
Assert.assertEquals(Order.DOUBLE, enol.getBond(0).getOrder());
Assert.assertEquals(Order.SINGLE, enol.getBond(1).getOrder());
Assert.assertEquals(Order.DOUBLE, enol.getBond(2).getOrder());
Assert.assertEquals(Order.SINGLE, enol.getBond(3).getOrder());
Assert.assertEquals(Order.SINGLE, enol.getBond(4).getOrder());
}

@Test
Expand Down
Expand Up @@ -203,11 +203,11 @@ public void xtestPyrrole() throws Exception {
//Assert.assertTrue(fbot.isOK(enol));

// now check whether it did the right thing
Assert.assertEquals(Order.DOUBLE, enol.getBond(0).getOrder());;
Assert.assertEquals(Order.SINGLE, enol.getBond(1).getOrder());;
Assert.assertEquals(Order.DOUBLE, enol.getBond(2).getOrder());;
Assert.assertEquals(Order.SINGLE, enol.getBond(3).getOrder());;
Assert.assertEquals(Order.SINGLE, enol.getBond(4).getOrder());;
Assert.assertEquals(Order.DOUBLE, enol.getBond(0).getOrder());
Assert.assertEquals(Order.SINGLE, enol.getBond(1).getOrder());
Assert.assertEquals(Order.DOUBLE, enol.getBond(2).getOrder());
Assert.assertEquals(Order.SINGLE, enol.getBond(3).getOrder());
Assert.assertEquals(Order.SINGLE, enol.getBond(4).getOrder());
}

@Test
Expand Down
Expand Up @@ -66,7 +66,7 @@ public static void add3DCoordinates1(IAtomContainer atomContainer) {
if (atom.getPoint3d() == null) {
List<IAtom> connectedAtoms = atomContainer.getConnectedAtomsList(atom);
if (connectedAtoms.size() == 1) {
IAtom refAtom = connectedAtoms.get(0);;
IAtom refAtom = connectedAtoms.get(0);
if (refAtom.getPoint3d() != null) {
refAtoms.addAtom(refAtom);
// store atoms with no coords and ref atoms in a
Expand Down Expand Up @@ -370,7 +370,7 @@ public static Point3d[] calculate3DCoordinates2(Point3d aPoint, Point3d bPoint,
Vector3d baxca = new Vector3d();
baxca.cross(ba, ca);
if (baxca.length() < 0.00000001) {
; // linear
// linear
} else if (nwanted == 1) {
newPoints = new Point3d[1];
Vector3d ax = new Vector3d(ba);
Expand Down
Expand Up @@ -546,7 +546,7 @@ public GIMatrix transpose() throws BadMatrixFormatException {
*/
public GIMatrix diagonal() throws BadMatrixFormatException {
if (m != n) throw new BadMatrixFormatException();
double[][] diagonal = new double[array.length][array[0].length];;
double[][] diagonal = new double[array.length][array[0].length];
for (int i = 0; i < m; i++)
for (int j = 0; j < n; j++) {
if (i == j)
Expand Down
Expand Up @@ -34,9 +34,9 @@

final class INChIContentProcessorTool {

private static final ILoggingTool logger = LoggingToolFactory.createLoggingTool(INChIContentProcessorTool.class); ;
private static final ILoggingTool logger = LoggingToolFactory.createLoggingTool(INChIContentProcessorTool.class);

public INChIContentProcessorTool() {}
public INChIContentProcessorTool() {}

/**
* Processes the content from the formula field of the INChI.
Expand Down
Expand Up @@ -252,8 +252,8 @@ public void doctypeDecl(String name, String publicId, String systemId) throws Ex
public void startDocument() {
chemFile = builder.newInstance(IChemFile.class);
chemSequence = builder.newInstance(IChemSequence.class);
chemModel = builder.newInstance(IChemModel.class);;
setOfMolecules = builder.newInstance(IAtomContainerSet.class);;
chemModel = builder.newInstance(IChemModel.class);
setOfMolecules = builder.newInstance(IAtomContainerSet.class);
}

@Override
Expand Down
Expand Up @@ -134,7 +134,7 @@ public void write(IChemObject object) throws CDKException {
} else {
throw new CDKException("Only Crystal objects can be read.");
}
};
}

// Private procedures

Expand Down
Expand Up @@ -59,7 +59,7 @@ public class VASPReader extends DefaultChemObjectReader {
private static final ILoggingTool logger = LoggingToolFactory.createLoggingTool(VASPReader.class);

// This variable is used to parse the input file
protected StringTokenizer st = new StringTokenizer("", ""); ;
protected StringTokenizer st = new StringTokenizer("", "");
protected String fieldVal;
protected int repVal = 0;

Expand Down
Expand Up @@ -256,7 +256,7 @@ public void write(IChemObject object) throws CDKException {
for (Integer rgrpNum : rGroupQuery.getRGroupDefinitions().keySet()) {
List<IRGroup> rgrpList = rGroupQuery.getRGroupDefinitions().get(rgrpNum).getRGroups();
if (rgrpList != null && rgrpList.size() != 0) {
rgpBlock.append("$RGP").append(LSEP);;
rgpBlock.append("$RGP").append(LSEP);
rgpBlock.append(MDLV2000Writer.formatMDLInt(rgrpNum, 4)).append(LSEP);

for (IRGroup rgroup : rgrpList) {
Expand Down
Expand Up @@ -142,7 +142,7 @@ public void testReadMapping() throws Exception {
@Test
public void testAgentParts() throws Exception {
try (InputStream in = this.getClass().getResourceAsStream("ethylesterification.mol");
MDLRXNV2000Reader rdr = new MDLRXNV2000Reader(in);) {
MDLRXNV2000Reader rdr = new MDLRXNV2000Reader(in)) {
IReaction reaction = rdr.read(new Reaction());
assertThat(reaction.getAgents().getAtomContainerCount(), is(1));
}
Expand Down
Expand Up @@ -85,7 +85,7 @@ public void testBug1571207() throws Exception {
String filename = "iterator/molV3000.mol";
logger.info("Testing: " + filename);
try (InputStream ins = this.getClass().getResourceAsStream(filename);
MDLV3000Reader reader = new MDLV3000Reader(ins);) {
MDLV3000Reader reader = new MDLV3000Reader(ins)) {
IAtomContainer m = reader.read(new AtomContainer());
reader.close();
Assert.assertNotNull(m);
Expand Down Expand Up @@ -115,7 +115,7 @@ public void testEmptyString() throws Exception {
@Test
public void testPseudoAtomLabels() throws Exception {
try (InputStream in = getClass().getResourceAsStream("pseudoatomsv3000.mol");
MDLV3000Reader reader = new MDLV3000Reader(in);) {
MDLV3000Reader reader = new MDLV3000Reader(in)) {
IAtomContainer molecule = DefaultChemObjectBuilder.getInstance().newInstance(IAtomContainer.class);
molecule = reader.read(molecule);
reader.close();
Expand Down
Expand Up @@ -71,7 +71,7 @@
public class Gaussian03Reader extends DefaultChemObjectReader {

private BufferedReader input;
private static final ILoggingTool logger = LoggingToolFactory.createLoggingTool(Gaussian03Reader.class); ;
private static final ILoggingTool logger = LoggingToolFactory.createLoggingTool(Gaussian03Reader.class);

public Gaussian03Reader(Reader reader) {
input = new BufferedReader(reader);
Expand Down

0 comments on commit 1ca0b58

Please sign in to comment.