Remove unchecked exceptions from method signatures

bp-alex · bp-alex · commit 8258e4871ecd · 2019-06-01T15:09:24.000+10:00
diff --git a/src/main/java/org/lmdbjava/Cursor.java b/src/main/java/org/lmdbjava/Cursor.java
@@ -355,7 +355,7 @@ public T val() {
     return kv.val();
   }
 
-  private void checkNotClosed() throws ClosedException {
+  private void checkNotClosed() {
     if (closed) {
       throw new ClosedException();
     }
diff --git a/src/main/java/org/lmdbjava/CursorIterator.java b/src/main/java/org/lmdbjava/CursorIterator.java
@@ -88,7 +88,7 @@ public Iterable<KeyVal<T>> iterable() {
   }
 
   @Override
-  public KeyVal<T> next() throws NoSuchElementException {
+  public KeyVal<T> next() {
     if (!hasNext()) {
       throw new NoSuchElementException();
     }
diff --git a/src/main/java/org/lmdbjava/ResultCodeMapper.java b/src/main/java/org/lmdbjava/ResultCodeMapper.java
@@ -56,9 +56,8 @@ private ResultCodeMapper() {
    * Checks the result code and raises an exception is not {@link #MDB_SUCCESS}.
    *
    * @param rc the LMDB result code
-   * @throws LmdbNativeException the resolved exception
    */
-  static void checkRc(final int rc) throws LmdbNativeException {
+  static void checkRc(final int rc) {
     switch (rc) {
       case MDB_SUCCESS:
         return;
diff --git a/src/main/java/org/lmdbjava/Txn.java b/src/main/java/org/lmdbjava/Txn.java
@@ -184,19 +184,19 @@ public T val() {
     return keyVal.val();
   }
 
-  void checkReadOnly() throws ReadOnlyRequiredException {
+  void checkReadOnly() {
     if (!readOnly) {
       throw new ReadOnlyRequiredException();
     }
   }
 
-  void checkReady() throws NotReadyException {
+  void checkReady() {
     if (state != READY) {
       throw new NotReadyException();
     }
   }
 
-  void checkWritesAllowed() throws ReadWriteRequiredException {
+  void checkWritesAllowed() {
     if (readOnly) {
       throw new ReadWriteRequiredException();
     }

Original file line number	Diff line number	Diff line change
`@@ -355,7 +355,7 @@ public T val() {`
`355`	`355`	`return kv.val();`
`356`	`356`	`}`
`357`	`357`
`358`		`- private void checkNotClosed() throws ClosedException {`
	`358`	`+ private void checkNotClosed() {`
`359`	`359`	`if (closed) {`
`360`	`360`	`throw new ClosedException();`
`361`	`361`	`}`
Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ public Iterable<KeyVal<T>> iterable() {`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`@Override`
`91`		`- public KeyVal<T> next() throws NoSuchElementException {`
	`91`	`+ public KeyVal<T> next() {`
`92`	`92`	`if (!hasNext()) {`
`93`	`93`	`throw new NoSuchElementException();`
`94`	`94`	`}`
Original file line number	Diff line number	Diff line change
`@@ -184,19 +184,19 @@ public T val() {`
`184`	`184`	`return keyVal.val();`
`185`	`185`	`}`
`186`	`186`
`187`		`- void checkReadOnly() throws ReadOnlyRequiredException {`
	`187`	`+ void checkReadOnly() {`
`188`	`188`	`if (!readOnly) {`
`189`	`189`	`throw new ReadOnlyRequiredException();`
`190`	`190`	`}`
`191`	`191`	`}`
`192`	`192`
`193`		`- void checkReady() throws NotReadyException {`
	`193`	`+ void checkReady() {`
`194`	`194`	`if (state != READY) {`
`195`	`195`	`throw new NotReadyException();`
`196`	`196`	`}`
`197`	`197`	`}`
`198`	`198`
`199`		`- void checkWritesAllowed() throws ReadWriteRequiredException {`
	`199`	`+ void checkWritesAllowed() {`
`200`	`200`	`if (readOnly) {`
`201`	`201`	`throw new ReadWriteRequiredException();`
`202`	`202`	`}`