diff options
author | Paul Phillips <paulp@improving.org> | 2012-10-03 13:54:30 -0700 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2012-10-03 13:58:14 -0700 |
commit | dee6a347335e9a4b42342664aa50b0cb217c00a9 (patch) | |
tree | 19054a14a7e0dea902156183c1fec41a386c9cf4 /src/compiler/scala/tools/nsc/symtab/classfile/Pickler.scala | |
parent | 5132659241e25b7695b3203fe0bc5c81c17b65d3 (diff) | |
download | scala-dee6a347335e9a4b42342664aa50b0cb217c00a9.tar.gz scala-dee6a347335e9a4b42342664aa50b0cb217c00a9.tar.bz2 scala-dee6a347335e9a4b42342664aa50b0cb217c00a9.zip |
Renamed hasSymbol to hasSymbolField.
Suggestion by retronym that the obvious implementation of
"hasSymbol" be called "hasSymbol" reminded me we have a method
called "hasSymbol" which does not have that implementation, and
which has burned us already with subtle bugginess. I think that
"hasSymbolField" is self-documenting.
Diffstat (limited to 'src/compiler/scala/tools/nsc/symtab/classfile/Pickler.scala')
-rw-r--r-- | src/compiler/scala/tools/nsc/symtab/classfile/Pickler.scala | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/compiler/scala/tools/nsc/symtab/classfile/Pickler.scala b/src/compiler/scala/tools/nsc/symtab/classfile/Pickler.scala index 5dbd22f23b..182338a0a1 100644 --- a/src/compiler/scala/tools/nsc/symtab/classfile/Pickler.scala +++ b/src/compiler/scala/tools/nsc/symtab/classfile/Pickler.scala @@ -68,7 +68,7 @@ abstract class Pickler extends SubComponent { return } - if (!t.isDef && t.hasSymbol && t.symbol.isTermMacro) { + if (!t.isDef && t.hasSymbolField && t.symbol.isTermMacro) { unit.error(t.pos, t.symbol.typeParams.length match { case 0 => "macro has not been expanded" case 1 => "this type parameter must be specified" @@ -235,7 +235,7 @@ abstract class Pickler extends SubComponent { private def putTree(tree: Tree): Unit = if (putEntry(tree)) { if (tree != EmptyTree) putType(tree.tpe) - if (tree.hasSymbol) + if (tree.hasSymbolField) putSymbol(tree.symbol) tree match { |