Skip to content

Commit 6bad505

Browse files
committed
sonarqube cleanup.
1 parent 46d5b98 commit 6bad505

4 files changed

Lines changed: 13 additions & 9 deletions

File tree

src/main/java/org/hdf5javalib/datatype/StringDatatype.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -248,6 +248,7 @@ public String toString() {
248248
*/
249249
@Override
250250
public void writeDefinitionToByteBuffer(ByteBuffer buffer) {
251+
// not implemented
251252
}
252253

253254
/**

src/main/java/org/hdf5javalib/hdffile/dataobjects/messages/HdfMessage.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ protected void writeMessageData(ByteBuffer buffer) {
105105
*/
106106
public abstract void writeMessageToByteBuffer(ByteBuffer buffer);
107107

108-
static class OBJECT_HEADER_PREFIX {
108+
public static class OBJECT_HEADER_PREFIX {
109109
final MessageType type;
110110
final int size;
111111
final int flags;

src/main/java/org/hdf5javalib/hdfjava/HdfDataset.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,11 @@ public boolean isDataset() {
3737

3838
@Override
3939
public void close() throws Exception {
40-
40+
// not implemented
4141
}
4242

4343
public void createAttribute(String attributeName, String attributeValue, HdfDataFile hdfDataFile) {
44+
// not implemented
4445
}
4546

4647
public Datatype getDatatype() {

src/main/java/org/hdf5javalib/utils/HdfDisplayUtils.java

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@
2929
public class HdfDisplayUtils {
3030
private static final Logger log = LoggerFactory.getLogger(HdfDisplayUtils.class);
3131
public static final String UNDEFINED = "<Undefined>";
32+
private static final String READ_EQUALS = " read = ";
33+
private static final String STREAM_EQUALS = " stream = ";
3234

3335
// Define a functional interface for actions that may need channel, dataset, and reader
3436
@FunctionalInterface
@@ -69,7 +71,7 @@ public static void displayFileAttr(Path filePath) {
6971
}
7072

7173
public static void displayFile(Path filePath) {
72-
processFile(filePath, (channel, dataSet, reader) -> displayData(channel, dataSet, reader));
74+
processFile(filePath, HdfDisplayUtils::displayData);
7375
}
7476

7577
public static void displayAttributes(HdfDataset dataSet) {
@@ -126,10 +128,10 @@ public static <T> void displayScalarData(SeekableByteChannel fileChannel, HdfDat
126128
TypedDataSource<T> dataSource = new TypedDataSource<>(fileChannel, hdfDataFile, dataSet, clazz);
127129

128130
T result = dataSource.readScalar();
129-
System.out.println(dataSet.getObjectName() + ":" + displayType(clazz, result) + " read = " + displayValue(result));
131+
System.out.println(dataSet.getObjectName() + ":" + displayType(clazz, result) + READ_EQUALS + displayValue(result));
130132

131133
result = dataSource.streamScalar().findFirst().orElseThrow();
132-
System.out.println(dataSet.getObjectName() + ":" + displayType(clazz, result) + " stream = " + displayValue(result));
134+
System.out.println(dataSet.getObjectName() + ":" + displayType(clazz, result) + STREAM_EQUALS + displayValue(result));
133135
}
134136

135137
/**
@@ -151,9 +153,9 @@ public static <T> void displayVectorData(SeekableByteChannel fileChannel, HdfDat
151153
TypedDataSource<T> dataSource = new TypedDataSource<>(fileChannel, hdfDataFile, dataSet, clazz);
152154

153155
T[] resultArray = dataSource.readVector();
154-
System.out.println(displayType(clazz, resultArray) + " read = " + displayValue(resultArray));
156+
System.out.println(displayType(clazz, resultArray) + READ_EQUALS + displayValue(resultArray));
155157

156-
System.out.print(displayType(clazz, resultArray) + " stream = [");
158+
System.out.print(displayType(clazz, resultArray) + STREAM_EQUALS);
157159
String joined = dataSource.streamVector()
158160
.map(HdfDisplayUtils::displayValue)
159161
.collect(Collectors.joining(", "));
@@ -180,9 +182,9 @@ public static <T> void displayMatrixData(SeekableByteChannel fileChannel, HdfDat
180182
TypedDataSource<T> dataSource = new TypedDataSource<>(fileChannel, hdfDataFile, dataSet, clazz);
181183

182184
T[][] resultArray = dataSource.readMatrix();
183-
System.out.println(displayType(clazz, resultArray) + " read = " + displayValue(resultArray));
185+
System.out.println(displayType(clazz, resultArray) + READ_EQUALS + displayValue(resultArray));
184186

185-
System.out.print(displayType(clazz, resultArray) + " stream = [");
187+
System.out.print(displayType(clazz, resultArray) + STREAM_EQUALS);
186188
String joined = dataSource.streamMatrix()
187189
.map(HdfDisplayUtils::displayValue)
188190
.collect(Collectors.joining(", "));

0 commit comments

Comments
 (0)