Spurious warning for implicit classes declared in package objects

[scabug]

package scala.reflect

package object api {
  implicit class PimpedExpr[T](expr: Universe # Expr[T]) {
    def runtimeEval: T = {
      println("hello, dear")
      expr.eval
    }
  }
}

Eval.scala:4: warning: it is not recommended to define classes/objects inside of package objects.
If possible, define class PimpedExpr in package api instead.

[scabug]

Imported From: https://issues.scala-lang.org/browse/SI-5809?orig=1
Reporter: @xeno-by
Affected Versions: 2.10.0-M3

[scabug]

@lrytz said:
for the moment, just drop the warning for implicit classes and hope for the best

[scabug]

@paulp said:
I moved the warning behind -Xlint a while before this ticket was opened. I don't think there's anything left to do here.

[scabug]

@som-snytt said:
"-Xlint would have told you not to do that".

I didn't want to be the first to comment. That was a couple of hours before I opened #5760.

[scabug]

@lrytz said (edited on May 23, 2012 2:47:37 PM UTC):
@paul ok, but the -Xlint warning is still wrong for implicit classes, they cannot be top-level.

[scabug]

@paulp said:
Right, that's true. I think I was subconsciously taking the position that imperfections in -Xlint output are "trees falling in a forest" but one never knows. If you assign this to me I'll fix it, or it's all yours.

[scabug]

@lrytz said:
who could say no to a volunteer! thanks.

[scabug]

@paulp said:
Fixed back in 006e12a119 .