Better CannotGetTimeException and CodecException with public constructors (#343)

Author: devgianlu

player/src/main/java/xyz/gianlu/librespot/player/codecs/Codec.java

@@ -121,12 +121,22 @@ public int decryptTimeMs() {
     }
 
     public static class CannotGetTimeException extends Exception {
-        CannotGetTimeException() {
+        public CannotGetTimeException(String message) {
+            super(message);
+        }
+
+        public CannotGetTimeException(String message, Throwable cause) {
+            super(message, cause);
         }
     }
 
     public static class CodecException extends Exception {
-        CodecException() {
+        public CodecException(String message) {
+            super(message);
+        }
+
+        public CodecException(String message, Throwable cause) {
+            super(message, cause);
         }
     }
 }

player/src/main/java/xyz/gianlu/librespot/player/codecs/Mp3Codec.java

@@ -84,7 +84,7 @@ public int readInternal(@NotNull OutputStream out) throws IOException {
 
     @Override
     public int time() throws CannotGetTimeException {
-        throw new CannotGetTimeException();
+        throw new CannotGetTimeException("No way to get time on MP3 stream");
     }
 
     @Override

player/src/main/java/xyz/gianlu/librespot/player/codecs/VorbisCodec.java

@@ -117,7 +117,7 @@ private void readHeader() throws IOException, CodecException {
                 }
 
                 if (joggStreamState.pagein(joggPage) == -1)
-                    throw new CodecException();
+                    throw new CodecException("Failed reading page");
 
                 if (joggStreamState.packetout(joggPacket) == -1)
                     throw new HoleInDataException();
@@ -133,7 +133,7 @@ private void readHeader() throws IOException, CodecException {
             buffer = joggSyncState.data;
 
             if (count == 0 && !finished)
-                throw new CodecException();
+                throw new CodecException("Buffer under-run");
         }
     }
 
@@ -153,7 +153,7 @@ public synchronized int readInternal(@NotNull OutputStream out) throws IOExcepti
             // Read more
         } else if (result == 1) {
             if (joggStreamState.pagein(joggPage) == -1)
-                throw new CodecException();
+                throw new CodecException("Failed reading page");
 
             if (joggPage.granulepos() == 0)
                 return -1;
@@ -244,8 +244,14 @@ public void close() throws IOException {
     }
 
     private static class NotVorbisException extends CodecException {
+        NotVorbisException() {
+            super("Data read is not vorbis data");
+        }
     }
 
     private static class HoleInDataException extends CodecException {
+        HoleInDataException() {
+            super("Hole in vorbis data");
+        }
     }
 }