Skip to content

Commit 189b7cb

Browse files
committed
Clean some code
1 parent 2543519 commit 189b7cb

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/org/beanio/internal/compiler/ParserFactorySupport.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -389,7 +389,7 @@ protected void initializeGroup(GroupConfig config) throws BeanIOConfigurationExc
389389

390390
protected void initializeGroupIteration(GroupConfig config, Property property) {
391391
// wrap the segment in an iteration
392-
Component aggregation = createRecordAggregation(config, property);
392+
Component aggregation = createRecordAggregation(config);
393393

394394
pushParser(aggregation);
395395
if (property != null || config.getTarget() != null) {
@@ -474,7 +474,7 @@ protected void initializeRecord(RecordConfig config) throws BeanIOConfigurationE
474474

475475
protected void initializeRecordIteration(RecordConfig config, Property property) {
476476
// wrap the segment in an iteration
477-
Component collection = createRecordAggregation(config, property);
477+
Component collection = createRecordAggregation(config);
478478

479479
pushParser(collection);
480480
if (property != null || config.getTarget() != null) {
@@ -586,7 +586,7 @@ protected void finalizeRecord(RecordConfig config, Record record) {
586586

587587
}
588588

589-
private Property findTarget(Component segment, String name) {
589+
private static Property findTarget(Component segment, String name) {
590590
Component c = findDescendant("value", segment, name);
591591
if (c == null) {
592592
throw new BeanIOConfigurationException("Descendant value '" + name + "' not found");
@@ -602,7 +602,7 @@ private Property findTarget(Component segment, String name) {
602602
return property;
603603
}
604604

605-
private Component findDescendant(String type, Component c, String name) {
605+
private static Component findDescendant(String type, Component c, String name) {
606606
if (name.equals(c.getName())) {
607607
return c;
608608
}
@@ -1024,7 +1024,7 @@ else if (arrayType == null) {
10241024
* @return the created {@link RecordAggregation}
10251025
* @throws BeanIOConfigurationException
10261026
*/
1027-
protected RecordAggregation createRecordAggregation(PropertyConfig config, Property property)
1027+
protected RecordAggregation createRecordAggregation(PropertyConfig config)
10281028
throws BeanIOConfigurationException
10291029
{
10301030
boolean isMap = false;
@@ -1049,7 +1049,7 @@ protected RecordAggregation createRecordAggregation(PropertyConfig config, Prope
10491049
}
10501050

10511051
// create the appropriate iteration type
1052-
RecordAggregation aggregation;;
1052+
RecordAggregation aggregation;
10531053
if (collectionType == TypeUtil.ARRAY_TYPE) {
10541054
aggregation = new RecordArray();
10551055
}

0 commit comments

Comments
 (0)