[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [aspectj-users] declare warning and if/etc now fixed.


Hi Philip,

Bug 51320 is for the ClassCastException you mention and I've just checked
in the fix.  Following the discussion on the dev list - the fix means
that any use of this/target/args/if/cflow/cflowbelow in a deow
statement will produce a compile time error.

This is as per the AspectJ docs (see end of this html page):

http://dev.eclipse.org/viewcvs/indextech.cgi/~checkout~/aspectj-home/doc/progguide/semantics-declare.html#d0e6499


Andy.

---
Andy Clement
AspectJ/AJDT Development.




Philip <philipvanbogaert@xxxxxxxxxx>
Sent by: aspectj-users-admin@xxxxxxxxxxx

24/02/2004 22:00

Please respond to
aspectj-users

To
aspectj-users@xxxxxxxxxxx
cc
Subject
Re: [aspectj-users] declare warning





I'm still convinsed that cflow could be implemented staticly but that doesn't
mean that it should. Neither way it should generate a compiler error.

a bug ?

following fragment causes a ClassCastException aspectj compiler
declare warning:if(true):"Pigs can fly";

So i suspect that the static keywords aren't checked in the compiler of ajdt
plugin (i've heared somewhere that it uses a beta version of aspectj)
eclipse: 2.1.2
ui: 0.6.4
tools: 1.1.4

Greetz,
Philip

Op dinsdag 24 februari 2004 17:00, schreef EyÃun Nielsen:
> Just wondering... Halting problem? ;-)
>
> ----
>  From "Aspect-Oriented Programming with AspectJ" by Ivan Kiselev, page 199:
> "...
> Because there is no runtime context at the compile time, the statically
> determinable pointcuts must not be, and cannot be, expressed in terms of
> the following primitive pointcuts that depend on runtime information to
> work.
>
> this
> target
> args
> cflow
> cflowbelow
> if
>
> ..."
>
> Regards,
> -eyÃun
>
> philipvanbogaert@xxxxxxxxxx wrote:
> >following code fragment doesn't seem's to work correctly,
> >i'm trying to give a warning for each joinpoint that isn't in the flow
> > hierachy of my main method.
> >
> >declare warning: !cflow(execution(void *.main(String[]))) : "Unreachable
> >code";
> >
> >Greetz,
> >Philip
> >
> >--------------------------------------------------------------------------
> >------ Tiscali ADSL: 19,50 â/maand, gedurende 3 maand! Snel Internet, en
> > voor iedereen. http://reg.tiscali.be/default.asp?lg=nl
> >
> >
> >
> >_______________________________________________
> >aspectj-users mailing list
> >aspectj-users@xxxxxxxxxxx
> >http://dev.eclipse.org/mailman/listinfo/aspectj-users
>
> _______________________________________________
> aspectj-users mailing list
> aspectj-users@xxxxxxxxxxx
> http://dev.eclipse.org/mailman/listinfo/aspectj-users

_______________________________________________
aspectj-users mailing list
aspectj-users@xxxxxxxxxxx
http://dev.eclipse.org/mailman/listinfo/aspectj-users