You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mnemonic.apache.org by "Wang, Gang (JIRA)" <ji...@apache.org> on 2017/01/26 21:59:24 UTC

[jira] [Closed] (MNEMONIC-180) Missing throw for NoSuchElementException();

     [ https://issues.apache.org/jira/browse/MNEMONIC-180?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Wang, Gang closed MNEMONIC-180.
-------------------------------

> Missing throw for NoSuchElementException();
> -------------------------------------------
>
>                 Key: MNEMONIC-180
>                 URL: https://issues.apache.org/jira/browse/MNEMONIC-180
>             Project: Mnemonic
>          Issue Type: Bug
>            Reporter: Jaechang Nam
>            Assignee: Wang, Gang
>            Priority: Trivial
>             Fix For: 0.5.0-incubating
>
>
> In recent snapshot from Github, it seems there is an exception (new NoSuchElementException) that is not thrown. (mnemonic-collections/src/main/java/org/apache/mnemonic/collections/DurableSinglyLinkedList.java)
> {code}
> 162     @Override
> 163     public E next() {
> 164       if (null == next) {
> 165         new NoSuchElementException();
> 166       }
> 167       E ret = next.getItem();
> 168       next = next.getNext();
> 169       return ret;
> 170     }
> {code}
> Should it be as follows?
> {code}
> ...
> 164       if (null == next) {
> 165         throw new NoSuchElementException();
> 166       }
> ...
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)