diff options
author | Eugene Burmako <xeno.by@gmail.com> | 2012-08-10 01:17:43 +0200 |
---|---|---|
committer | Eugene Burmako <xeno.by@gmail.com> | 2012-08-10 10:59:24 +0200 |
commit | e01ec05a1e7a937cfed13cb3a85f70e26950fdbf (patch) | |
tree | aab0582af3c3fe0586effe93c794bebeef26af18 /src/reflect/scala/reflect/api/Symbols.scala | |
parent | 0e2080e8bf1c54cac59c5b0c53a93eb0de009d3b (diff) | |
download | scala-e01ec05a1e7a937cfed13cb3a85f70e26950fdbf.tar.gz scala-e01ec05a1e7a937cfed13cb3a85f70e26950fdbf.tar.bz2 scala-e01ec05a1e7a937cfed13cb3a85f70e26950fdbf.zip |
refines api.FlagSets
In the current vision of flags, they should only be used to construct trees,
because tests are subsumed by the numerous isXXX methods that we have.
Hence we need to remove `hasFlag` and the flags that don't make sense being
added to tree modifiers manually.
A good example of that is MODULE. There's an isModule check, so hasFlag MODULE
is redundant. On the other hand, you don't need MODULE to construct a ModuleDef
so it's redundant in this department as well.
Diffstat (limited to 'src/reflect/scala/reflect/api/Symbols.scala')
-rw-r--r-- | src/reflect/scala/reflect/api/Symbols.scala | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/reflect/scala/reflect/api/Symbols.scala b/src/reflect/scala/reflect/api/Symbols.scala index 448382973a..1efffc910f 100644 --- a/src/reflect/scala/reflect/api/Symbols.scala +++ b/src/reflect/scala/reflect/api/Symbols.scala @@ -110,10 +110,10 @@ trait Symbols extends base.Symbols { self: Universe => * * The java access levels translate as follows: * - * java private: hasFlag(PRIVATE) && (privateWithin == NoSymbol) - * java package: !hasFlag(PRIVATE | PROTECTED) && (privateWithin == enclosingPackage) - * java protected: hasFlag(PROTECTED) && (privateWithin == enclosingPackage) - * java public: !hasFlag(PRIVATE | PROTECTED) && (privateWithin == NoSymbol) + * java private: isPrivate && (privateWithin == NoSymbol) + * java package: !isPrivate && !isProtected && (privateWithin == enclosingPackage) + * java protected: isProtected && (privateWithin == enclosingPackage) + * java public: !isPrivate && !isProtected && (privateWithin == NoSymbol) */ def privateWithin: Symbol |