Switch expressions are exhaustive, which is exactly what we want to
do here.
Change-Id: I32312c9ee8ac0f4e5da919b2b1d384a37cbadaad
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
* terms of the Eclipse Public License v1.0 which accompanies this distribution,
* and is available at http://www.eclipse.org/legal/epl-v10.html
*/
* terms of the Eclipse Public License v1.0 which accompanies this distribution,
* and is available at http://www.eclipse.org/legal/epl-v10.html
*/
package org.opendaylight.netconf.nettyutil.handler;
import io.netty.buffer.ByteBuf;
package org.opendaylight.netconf.nettyutil.handler;
import io.netty.buffer.ByteBuf;
import org.slf4j.LoggerFactory;
public final class FramingMechanismHandlerFactory {
import org.slf4j.LoggerFactory;
public final class FramingMechanismHandlerFactory {
private static final Logger LOG = LoggerFactory.getLogger(FramingMechanismHandlerFactory.class);
private FramingMechanismHandlerFactory() {
// not called - private constructor for utility class
}
private static final Logger LOG = LoggerFactory.getLogger(FramingMechanismHandlerFactory.class);
private FramingMechanismHandlerFactory() {
// not called - private constructor for utility class
}
- public static MessageToByteEncoder<ByteBuf> createHandler(FramingMechanism framingMechanism) {
+ public static MessageToByteEncoder<ByteBuf> createHandler(final FramingMechanism framingMechanism) {
LOG.debug("{} framing mechanism was selected.", framingMechanism);
LOG.debug("{} framing mechanism was selected.", framingMechanism);
- if (framingMechanism == FramingMechanism.EOM) {
- return new EOMFramingMechanismEncoder();
- } else {
- return new ChunkedFramingMechanismEncoder();
- }
+ return switch (framingMechanism) {
+ case CHUNK -> new ChunkedFramingMechanismEncoder();
+ case EOM -> new EOMFramingMechanismEncoder();
+ };