parsing "default scope"

[GoogleCodeExporter]

parsing "default scope"

[GoogleCodeExporter]

This is because the class above is declared "package-private". "package-private"
means neither "private", nor "public", nor "protected" modifier is used. 
ModifierSet
can only check if a modifier was actually used in the parsed code.

Original comment by [email protected] on 15 Jan 2010 at 10:06

[GoogleCodeExporter]

Thanks michaelkoch6.
It seems rodrigo had a little mistake. Am I right?
I will close this issue until this mistake be demystified.

Original comment by [email protected] on 15 Jan 2010 at 3:14

• Changed state: Invalid

[GoogleCodeExporter]

Hi,
i think it should be useful to have that "package private" modifier in the 
parser.
i'm making a diff. tool and out of the box i can't know if the modifier has 
changed 
from package private to just private (or other).
To solve the issue i had to implement a exhaustive if (if it's not private, not 
public, etc. then it should be package private). It's not a very clean 
solution, it 
would be nicer if the parser could answer "ModifierSet.isPackagePrivate(...)".
Just my opinion ;)

Original comment by [email protected] on 15 Jan 2010 at 3:22

[GoogleCodeExporter]

package private means the lack of modifiers. This is why there is no flag for 
this.
If the method "ModifierSet.isPackagePrivate(...)" existed, it's implementation 
should
be composed by the same ifs you've done in your code. 

I agree with you that in some situations it could be useful. But it shouldn't be
implemented in the class "ModifierSet". Maybe in a helper class.

Original comment by [email protected] on 15 Jan 2010 at 3:31

[GoogleCodeExporter]

Well, maybe a method "hasModifiers()" or something like that make more sence, i 
see 
the point for not having a package private modifier (as modifiers do that, 
explicitly change the default package private behavior). 

Original comment by [email protected] on 15 Jan 2010 at 3:43