Skip to content

Commit

Permalink
Warnings cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
cowtowncoder committed Nov 30, 2023
1 parent e7c852b commit 769a4c4
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 20 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -910,7 +910,7 @@ public CsvSchema(Column[] columns, int features,
if (_columns.length == 0) {
_columnsByName = Collections.emptyMap();
} else {
_columnsByName = new LinkedHashMap<String,Column>(4 + _columns.length);
_columnsByName = new LinkedHashMap<>(4 + _columns.length);
for (Column c : _columns) {
_columnsByName.put(c.getName(), c);
}
Expand Down Expand Up @@ -959,7 +959,7 @@ protected CsvSchema(CsvSchema base, Column[] columns)
if (_columns.length == 0) {
_columnsByName = Collections.emptyMap();
} else {
_columnsByName = new LinkedHashMap<String,Column>(4 + _columns.length);
_columnsByName = new LinkedHashMap<>(4 + _columns.length);
for (Column c : _columns) {
_columnsByName.put(c.getName(), c);
}
Expand Down Expand Up @@ -1267,7 +1267,7 @@ public CsvSchema withAnyPropertyName(String name) {
*/
public CsvSchema sortedBy(String... columnNames)
{
LinkedHashMap<String,Column> map = new LinkedHashMap<String,Column>();
LinkedHashMap<String,Column> map = new LinkedHashMap<>();
for (String colName : columnNames) {
Column col = _columnsByName.get(colName);
if (col != null) {
Expand All @@ -1287,7 +1287,7 @@ public CsvSchema sortedBy(String... columnNames)
* @since 2.4
*/
public CsvSchema sortedBy(Comparator<String> cmp) {
TreeMap<String,Column> map = new TreeMap<String,Column>(cmp);
TreeMap<String,Column> map = new TreeMap<>(cmp);
for (Column col : _columns) {
map.put(col.getName(), col);
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,26 +1,22 @@
package com.fasterxml.jackson.dataformat.javaprop.dos;

import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.core.JsonToken;
import com.fasterxml.jackson.core.StreamReadConstraints;
import com.fasterxml.jackson.core.exc.StreamConstraintsException;

import com.fasterxml.jackson.databind.ObjectMapper;

import com.fasterxml.jackson.dataformat.javaprop.JavaPropsFactory;
import com.fasterxml.jackson.dataformat.javaprop.ModuleTestBase;

import java.io.IOException;

public class DeepNestParserTest extends ModuleTestBase {

public void testDeeplyNestedData() throws IOException {
public void testDeeplyNestedData() throws Exception {
final int depth = 1500;
final String doc = genDeeplyNestedData(depth);
final ObjectMapper mapper = newPropertiesMapper();
try (JsonParser jp = mapper.createParser(doc)) {
JsonToken jt;
while ((jt = jp.nextToken()) != null) {

}
try (JsonParser p = mapper.createParser(doc)) {
while (p.nextToken() != null) { }
fail("expected StreamConstraintsException");
} catch (StreamConstraintsException e) {
String exceptionPrefix = String.format("Document nesting depth (%d) exceeds the maximum allowed",
Expand All @@ -30,18 +26,15 @@ public void testDeeplyNestedData() throws IOException {
}
}

public void testDeeplyNestedDataWithUnconstrainedMapper() throws IOException {
public void testDeeplyNestedDataWithUnconstrainedMapper() throws Exception {
final int depth = 1500;
final String doc = genDeeplyNestedData(depth);
final JavaPropsFactory factory = JavaPropsFactory.builder()
.streamReadConstraints(StreamReadConstraints.builder().maxNestingDepth(Integer.MAX_VALUE).build())
.build();
final ObjectMapper mapper = propertiesMapperBuilder(factory).build();
try (JsonParser jp = mapper.createParser(doc)) {
JsonToken jt;
while ((jt = jp.nextToken()) != null) {

}
try (JsonParser p = mapper.createParser(doc)) {
while (p.nextToken() != null) { }
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
import java.time.OffsetDateTime;
import java.util.Arrays;

@SuppressWarnings("OctalInteger")
public class ParserTest extends TomlMapperTestBase {
private static final ObjectMapper TOML_MAPPER = newTomlMapper();
private static final ObjectMapper jsonMapper = JsonMapper.builder()
Expand Down

0 comments on commit 769a4c4

Please sign in to comment.