Merge pull request ably#613 from ably/feature/checkstyle-avoid-star-import

Checkstyle: AvoidStarImport

Author: Quintin Willison

config/checkstyle/checkstyle.xml

@@ -11,5 +11,6 @@
 	<module name="TreeWalker">
 		<module name="RedundantModifier"/>
 		<module name="UnusedImports"/>
+		<module name="AvoidStarImport"/>
 	</module>
 </module>

lib/src/main/java/io/ably/lib/http/HttpCore.java

@@ -1,8 +1,14 @@
 package io.ably.lib.http;
 
-import java.io.*;
+import java.io.ByteArrayOutputStream;
+import java.io.InputStream;
+import java.io.IOException;
+import java.io.OutputStream;
 import java.lang.reflect.Field;
-import java.net.*;
+import java.net.HttpURLConnection;
+import java.net.InetSocketAddress;
+import java.net.Proxy;
+import java.net.URL;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;

lib/src/main/java/io/ably/lib/http/HttpHelpers.java

@@ -3,7 +3,10 @@
 import java.io.IOException;
 import java.net.URL;
 
-import io.ably.lib.types.*;
+import io.ably.lib.types.AblyException;
+import io.ably.lib.types.Callback;
+import io.ably.lib.types.ErrorInfo;
+import io.ably.lib.types.Param;
 
 import static io.ably.lib.http.HttpUtils.buildURL;
 

lib/src/main/java/io/ably/lib/push/PushBase.java

@@ -2,11 +2,19 @@
 
 import com.google.gson.JsonElement;
 import com.google.gson.JsonObject;
-import io.ably.lib.http.*;
+import io.ably.lib.http.BasePaginatedQuery;
+import io.ably.lib.http.Http;
+import io.ably.lib.http.HttpCore;
+import io.ably.lib.http.HttpScheduler;
+import io.ably.lib.http.HttpUtils;
 import io.ably.lib.realtime.CompletionListener;
 import io.ably.lib.rest.AblyBase;
 import io.ably.lib.rest.DeviceDetails;
-import io.ably.lib.types.*;
+import io.ably.lib.types.AblyException;
+import io.ably.lib.types.AsyncPaginatedResult;
+import io.ably.lib.types.Callback;
+import io.ably.lib.types.PaginatedResult;
+import io.ably.lib.types.Param;
 import io.ably.lib.util.Serialisation;
 import io.ably.lib.util.StringUtils;
 

lib/src/main/java/io/ably/lib/realtime/Presence.java

@@ -14,8 +14,16 @@
 import io.ably.lib.types.PresenceSerializer;
 import io.ably.lib.types.ProtocolMessage;
 import io.ably.lib.util.Log;
-
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.EnumMap;
+import java.util.EnumSet;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
 
 /**
  * A class that provides access to presence operations and state for the

lib/src/main/java/io/ably/lib/rest/AblyBase.java

@@ -15,7 +15,22 @@
 import io.ably.lib.http.PaginatedQuery;
 import io.ably.lib.platform.Platform;
 import io.ably.lib.push.Push;
-import io.ably.lib.types.*;
+import io.ably.lib.types.AblyException;
+import io.ably.lib.types.AsyncHttpPaginatedResponse;
+import io.ably.lib.types.AsyncPaginatedResult;
+import io.ably.lib.types.Callback;
+import io.ably.lib.types.ChannelOptions;
+import io.ably.lib.types.ClientOptions;
+import io.ably.lib.types.ErrorInfo;
+import io.ably.lib.types.HttpPaginatedResponse;
+import io.ably.lib.types.Message;
+import io.ably.lib.types.MessageSerializer;
+import io.ably.lib.types.PaginatedResult;
+import io.ably.lib.types.Param;
+import io.ably.lib.types.PublishResponse;
+import io.ably.lib.types.ReadOnlyMap;
+import io.ably.lib.types.Stats;
+import io.ably.lib.types.StatsReader;
 import io.ably.lib.util.Crypto;
 import io.ably.lib.util.InternalMap;
 import io.ably.lib.util.Log;

lib/src/main/java/io/ably/lib/transport/ConnectionManager.java

@@ -3,7 +3,12 @@
 import io.ably.lib.debug.DebugOptions;
 import io.ably.lib.debug.DebugOptions.RawProtocolListener;
 import io.ably.lib.http.HttpHelpers;
-import io.ably.lib.realtime.*;
+import io.ably.lib.realtime.AblyRealtime;
+import io.ably.lib.realtime.Channel;
+import io.ably.lib.realtime.CompletionListener;
+import io.ably.lib.realtime.Connection;
+import io.ably.lib.realtime.ConnectionState;
+import io.ably.lib.realtime.ConnectionStateListener;
 import io.ably.lib.realtime.ConnectionStateListener.ConnectionStateChange;
 import io.ably.lib.transport.ITransport.ConnectListener;
 import io.ably.lib.transport.ITransport.TransportParams;
@@ -15,8 +20,12 @@
 import io.ably.lib.types.ProtocolSerializer;
 import io.ably.lib.util.Log;
 import io.ably.lib.transport.NetworkConnectivity.NetworkConnectivityListener;
-
-import java.util.*;
+import java.util.ArrayDeque;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Map;
 
 public class ConnectionManager implements ConnectListener {
 

lib/src/main/java/io/ably/lib/types/Message.java

@@ -3,8 +3,14 @@
 import java.io.IOException;
 import java.lang.reflect.Type;
 import java.util.Collection;
-
-import com.google.gson.*;
+import com.google.gson.JsonArray;
+import com.google.gson.JsonDeserializer;
+import com.google.gson.JsonDeserializationContext;
+import com.google.gson.JsonElement;
+import com.google.gson.JsonObject;
+import com.google.gson.JsonParseException;
+import com.google.gson.JsonSerializationContext;
+import com.google.gson.JsonSerializer;
 import io.ably.lib.util.Serialisation;
 import org.msgpack.core.MessageFormat;
 import org.msgpack.core.MessagePacker;

lib/src/main/java/io/ably/lib/types/PresenceMessage.java

@@ -2,8 +2,14 @@
 
 import java.io.IOException;
 import java.lang.reflect.Type;
-
-import com.google.gson.*;
+import com.google.gson.JsonArray;
+import com.google.gson.JsonDeserializer;
+import com.google.gson.JsonDeserializationContext;
+import com.google.gson.JsonElement;
+import com.google.gson.JsonObject;
+import com.google.gson.JsonParseException;
+import com.google.gson.JsonSerializationContext;
+import com.google.gson.JsonSerializer;
 import io.ably.lib.util.Serialisation;
 import org.msgpack.core.MessageFormat;
 import org.msgpack.core.MessagePacker;