fixed

On 01/22/2013 09:33 AM, Mirco Dotta wrote:
>
> In 
> org.scala-ide.sdt.core/src/scala/tools/eclipse/lexical/ScalaCodeScanner.scala:
>
> >
> > -    val result = token.tokenType match {
> > -      case WS                  => Token.WHITESPACE
> > -      case EOF                 => Token.EOF
> > -      case _ if isMacro(token) => getToken(ScalaSyntaxClasses.KEYWORD)
> > -      case _                   => getToken(ScalariformToSyntaxClass(token))
> > +    if (ranges.isEmpty) {
> > +      return
>
> I believe we can avoid the |return| here (by inverting the condition 
> and moving inside the if-expression the code that follows)
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/scala-ide/scala-ide/pull/279/files#r2722894>.
>