Skip to content

Commit 072b330

Browse files
authored
Merge pull request #2490 from matsim-org/emissionsBugfix
add a break for a switch in emissions contrib
2 parents 9af4dcb + 0549379 commit 072b330

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

contribs/emissions/src/main/java/org/matsim/contrib/emissions/ColdEmissionHandler.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,9 @@ static void handleNullVehicleECG(Id<Vehicle> vehicleId, EmissionsConfigGroup emi
138138
"No vehicle defined for id " + vehicleId + ". The vehicle will be ignored." );
139139
noVehWarnCnt++;
140140
if ( noVehWarnCnt == 10 ) logger.warn( Gbl.FUTURE_SUPPRESSED );
141-
}
141+
}
142+
break;
143+
// yyyy I found the above without the "break". There is https://github.com/matsim-org/matsim-code-examples/issues/910, according to which it seems that this is a bug. There should be a testcase, but I guess there ain't. kai, apr'22
142144
default:
143145
throw new RuntimeException( "Not yet implemented. Aborting..." );
144146
}

0 commit comments

Comments
 (0)