Re: [aspectj-users] AJDT 1.6.2 not dealing with private members on priv

[Andrew Eisenberg <andrew@xxxxxxxxxxxx>]

Are you not getting any [Xlint:invalidAbsoluteTypeName] warnings?
Just tested on this:

public aspect NoImports {

    pointcut f() : execution(List.new());
}

And got a warning.  Do you have something simple that is not working for you?

On Tue, Jan 27, 2009 at 3:07 AM, Simone Gianni <simoneg@xxxxxxxxxx> wrote:
> Hi Andrew,
> we noticed yesterday that 1.6.2 is not giving a warning when a pointcut is
> expressed on a type which is not imported. Previous versions used to give a
> warning about unresolved types. I think this misbehavior could be connected
> with the reconciling mechanism as well.
>
> Simone
>
> Andrew Eisenberg wrote:
>>
>> Hi,
>>
>> I raised bug 262404 to track this issue.
>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=262404
>>
>> What you are seeing are reconciling errors (ie- eager parsing).  The
>> compiler is doing the right thing, but the editor is not recognizing
>> the privileged keyword.  For 1.6.2, we revamped the reconciling
>> mechanism so that aspects can use an eager parser and check for errors
>> as you type, but there is still some syntax that is not fully handled.
>>
>> On Mon, Jan 26, 2009 at 5:57 AM, Neale Upstone <neale.upstone@xxxxxxxxxx>
>> wrote:
>>
>>>
>>> Hi,
>>>
>>>
>>>
>>> I've got two Eclipse 3.4.1 installs, each with the AJDT zip expanded to
>>> dropins.  (I'm not using JDT Weaving – I had problems trying to install
>>> via
>>> the 3.4 release builds update site)
>>>
>>>
>>>
>>> The zips are the 1.6.2 and 1.6.1a release builds from the downloads page.
>>>
>>>
>>>
>>> Code to access a private member of the target works in 1.6.1a, but gives
>>> "The field x is not visible" in the editor (strangely not in the Problems
>>> pane) under 1.6.2.
>>>
>>>
>>>
>>> After a clean build, the code does run correctly and the aspect does seem
>>> to
>>> be correctly woven (i.e. the advice is seen to be applied).
>>>
>>>
>>>
>>> Is anyone aware of this.  It seems like there should be a regression test
>>> on
>>> something like this.
>>>
>>>
>>>
>>> Cheers,
>>>
>>>
>>>
>>> Neale
>>>
>>>
>>>
>>>
>>>
>>> **********************************************************************
>>> IMPORTANT NOTICE.
>>> Confidentiality:  This e-mail and its attachments are intended for the
>>> above
>>> named only and may be confidential.  If they have come to you in error
>>> you
>>> must take no action based on them, nor must you copy or show them to
>>> anyone;
>>> please reply to this e-mail and highlight the error.
>>> Security Warning:  Please note that this e-mail has been created in the
>>> knowledge that Internet e-mail is not a 100% secure communications
>>> medium.
>>> We advise that you understand and observe this lack of security when
>>> e-mailing us.
>>> Viruses:  Although we have taken steps to ensure that this e-mail and
>>> attachments are free from any virus, we advise that in keeping with good
>>> computing practice the recipient should ensure they are actually virus
>>> free.
>>> Monitoring and Scanning:  Cambridge Cognition has monitoring and scanning
>>> systems in place in relation to emails sent and received to: monitor /
>>> record business communications; prevent and detect crime; investigate the
>>> use of the Company's internal and external email system; and provide
>>> evidence of compliance with business practices.
>>>
>>> Cambridge Cognition Limited
>>> Company Registration Number 4338746
>>> Registered address:
>>> Tunbridge Court
>>> Tunbridge Lane
>>> Bottisham
>>> Cambridge
>>> CB25 9TU
>>> UK
>>> **********************************************************************
>>>
>>>
>>> _______________________________________________
>>> aspectj-users mailing list
>>> aspectj-users@xxxxxxxxxxx
>>> https://dev.eclipse.org/mailman/listinfo/aspectj-users
>>>
>>>
>>>
>>
>> _______________________________________________
>> aspectj-users mailing list
>> aspectj-users@xxxxxxxxxxx
>> https://dev.eclipse.org/mailman/listinfo/aspectj-users
>>
>
>
> --
> Simone Gianni            CEO Semeru s.r.l.           Apache Committer
> http://www.simonegianni.it/
>
> _______________________________________________
> aspectj-users mailing list
> aspectj-users@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/aspectj-users
>