Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] CPPClassType.getVisibility(IBinding)




On Tue, Oct 1, 2013 at 5:12 AM, Andrew Eidsness <eclipse@xxxxxxxxxx> wrote:
I'm trying to figure out why this method doesn't check if the argument is an ICPPMember and then use
ICPPMember#getVisibility.  This seems to lead to, at least the potential for, inconsistent results with cases like
PDOMCPPAnnotation#encodeAnnotation (which does check instanceof, etc.).

A nested type is not a ICPPMember but may be used as an argument of  CPPClassType.getVisibility(IBinding). See https://bugs.eclipse.org/bugs/show_bug.cgi?id=402878 for more information.

To be clear, I haven't traced this to a specific problem in the core CDT, I'm just trying to figure out the motivation
behind this approach.

It does cause problems in some non-standard extensions which worked in 8.1.  Those extensions used internal types and
shouldn't be expected to continue working.  I'm trying to figure out the design goals to reduce the chance of the fix to
the non-standard extensions breaking in future releases.

Thanks,
-Andrew

-sergey 
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev


Back to the top