done

On Tue 22 Jan 2013 04:11:56 PM CET, Mirco Dotta wrote:
> In
> org.scala-ide.sdt.core/src/scala/tools/eclipse/semantichighlighting/classifier/SafeSymbol.scala:
>
> > @@ -104,6 +104,9 @@ trait SafeSymbol extends CompilerAccess with PimpedTrees {
> >      case ExistentialTypeTree(tpt, whereClauses) =>
> >        (tpt :: whereClauses).flatMap(safeSymbol)
> >
> > +    case LabelDef(name, _, _) if name.toString.matches("while\\$\\d+") =>
>
> What do you think of extracting the guard in a method, e.g.,
> |isWhileKeyword|? I feel this would clearly describe what is being
> checked.
>
> One more thing, it would be nice if you could use
> |global.nme.WHILE_PREFIX| (you'll have to escape the returned String)
> instead of |while\\$|
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scala-ide/scala-ide/pull/281/files#r2726628>.
>