dev@grizzly.java.net

Weird code in SelectorThread

From: Igor Minar <iiminar_at_gmail.com>
Date: Mon, 18 Jan 2010 02:37:42 -0800

Found this odd code in 1.9.18-k:

     public Selector getSelector(){
         if (selector != null) {
             return selector;
         } else {
             selectorHandler.getSelector();
         }

         return null;
     }


It basically means, if selector is not null return it, otherwise fetch
it from selectorHandler but don't store it anywhere, instead return
null.

Should it be:

     public Selector getSelector(){
         if (selector != null) {
             return selector;
         } else {
             return selector = selectorHandler.getSelector();
         }
     }