Skip to content

Commit 07aa5fb

Browse files
committed
Remove unnecesary code
Following hints from IDE: - remove ignored blank lines in javadoc - remove unused variables - inline variable - simplify expressions that are always true
1 parent ca71bb6 commit 07aa5fb

File tree

1 file changed

+2
-11
lines changed

1 file changed

+2
-11
lines changed

metafacture-biblio/src/main/java/org/metafacture/biblio/marc21/MarcXmlEncoder.java

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,6 @@ public final class MarcXmlEncoder extends DefaultStreamPipe<ObjectReceiver<Strin
4949
public static final boolean OMIT_XML_DECLARATION = false;
5050
public static final boolean ENSURE_CORRECT_MARC21_XML = false;
5151

52-
private static final String ROOT_OPEN = "<marc:collection xmlns:marc=\"http://www.loc.gov/MARC21/slim\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\" xsi:schemaLocation=\"http://www.loc.gov/MARC21/slim http://www.loc.gov/standards/marcxml/schema/MARC21slim.xsd\">";
53-
private static final String ROOT_CLOSE = "</marc:collection>";
54-
5552
private enum Tag {
5653

5754
collection(" xmlns%s=\"" + NAMESPACE + "\"%s"),
@@ -106,7 +103,6 @@ public String close(final Object[] args) {
106103
private static final int TAG_END = 3;
107104

108105
private final Encoder encoder = new Encoder();
109-
private final Marc21Decoder decoder = new Marc21Decoder();
110106
private final Marc21Encoder wrapper = new Marc21Encoder();
111107

112108
private DefaultStreamPipe<ObjectReceiver<String>> pipe;
@@ -115,6 +111,7 @@ public String close(final Object[] args) {
115111
* Creates an instance of {@link MarcXmlEncoder}.
116112
*/
117113
public MarcXmlEncoder() {
114+
final Marc21Decoder decoder = new Marc21Decoder();
118115
decoder.setEmitLeaderAsWhole(true);
119116

120117
wrapper
@@ -136,7 +133,6 @@ public void setEmitNamespace(final boolean emitNamespace) {
136133

137134
/**
138135
* Sets the flag to decide whether to omit the XML declaration.
139-
*
140136
* <strong>Default value: {@value #OMIT_XML_DECLARATION}</strong>
141137
*
142138
* @param currentOmitXmlDeclaration true if the XML declaration is omitted, otherwise
@@ -148,7 +144,6 @@ public void omitXmlDeclaration(final boolean currentOmitXmlDeclaration) {
148144

149145
/**
150146
* Sets the XML version.
151-
*
152147
* <strong>Default value: {@value #XML_VERSION}</strong>
153148
*
154149
* @param xmlVersion the XML version
@@ -159,7 +154,6 @@ public void setXmlVersion(final String xmlVersion) {
159154

160155
/**
161156
* Sets the XML encoding.
162-
*
163157
* <strong>Default value: {@value #XML_ENCODING}</strong>
164158
*
165159
* @param xmlEncoding the XML encoding
@@ -173,7 +167,6 @@ public void setXmlEncoding(final String xmlEncoding) {
173167
* If true, the input data is validated to ensure correct MARC21. Also the leader may be generated.
174168
* It acts as a wrapper: the input is piped to {@link org.metafacture.biblio.marc21.Marc21Encoder}, whose output is piped to {@link org.metafacture.biblio.marc21.Marc21Decoder}, whose output is piped to {@link org.metafacture.biblio.marc21.MarcXmlEncoder}.
175169
* This validation and treatment of the leader is more safe but comes with a performance impact.
176-
*
177170
* <strong>Default value: {@value #ENSURE_CORRECT_MARC21_XML}</strong>
178171
*
179172
* @param ensureCorrectMarc21Xml if true the input data is validated to ensure correct MARC21. Also the leader may be generated.
@@ -184,7 +177,6 @@ public void setEnsureCorrectMarc21Xml(final boolean ensureCorrectMarc21Xml) {
184177

185178
/**
186179
* Formats the resulting xml by indentation. Aka "pretty printing".
187-
*
188180
* <strong>Default value: {@value #PRETTY_PRINTED}</strong>
189181
*
190182
* @param formatted true if formatting is activated, otherwise false
@@ -247,7 +239,7 @@ private static class Encoder extends DefaultStreamPipe<ObjectReceiver<String>> {
247239
private String currentEntity = "";
248240

249241
private boolean emitNamespace = true;
250-
private Object[] namespacePrefix = new Object[]{emitNamespace ? NAMESPACE_PREFIX : EMPTY};
242+
private Object[] namespacePrefix = new Object[]{NAMESPACE_PREFIX};
251243

252244
private int indentationLevel;
253245
private boolean formatted = PRETTY_PRINTED;
@@ -463,7 +455,6 @@ private void writeTagLeader(final Function<Object[], String> function) {
463455
writeRawLeader(function.apply(namespacePrefix));
464456
}
465457

466-
467458
private void prettyPrintIndentation() {
468459
if (formatted) {
469460
final String prefix = String.join("", Collections.nCopies(indentationLevel, INDENT));

0 commit comments

Comments
 (0)