Unused client options manipulation

[ctron]

In the constructor of the MQTT client, it looks like some changes in the past have changed the originally intended behavior:

vertx-mqtt/src/main/java/io/vertx/mqtt/impl/MqttClientImpl.java

Lines 156 to 165 in 9a91ad8

	public MqttClientImpl(Vertx vertx, MqttClientOptions options) {
	// copy given options
	NetClientOptions netClientOptions = new NetClientOptions(options);
	netClientOptions.setIdleTimeout(DEFAULT_IDLE_TIMEOUT);
	this.vertx = (VertxInternal) vertx;
	this.options = new MqttClientOptions(options);
	this.keepAliveTimeout = ((options.getKeepAliveInterval() * 1000) * 3) / 2;
	}

From what I see, the netClientOptions gets copied, manipulated, but then not used at all.