| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
| |
ClasscastException on concretization of if(false)
(i.e. you can't use if/target/args/cflow/cflowbelow/this in deow - you get an error if you try)
|
| |
|
|
| |
Polymorphic ITD fails in CVS HEAD (From ajdt 1.1.6)
|
| |
|
|
|
|
|
|
|
| |
Advice calling protected super method causing java.lang.VerifyError 'Bad access to protected data'
Also expanded test to cover protected field access as well as methods
Fix required getting the correct receiver type for both field access and method
calls to correspond to Java's complicated rules for accessing protected
members (JLSv2 6.6.2 and mentioned in passing in JVMv2 5.4.4)
|
| |
|
|
|
| |
Better binary compatibility for advice method names
- I've run the tests a thousand times and they all pass, I'm still nervous about this first big commit though *gulp*
|
| |
|
|
| |
effect of an after returning type incompatible with a join point return type
|
| |
|
|
| |
CatchClauseSignature has broken operation
|
| |
|
|
| |
duplicate warning or second join point for constructor-execution
|
| |
|
|
| |
NPE When compiling intertype declaration
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
Erroneous exception conversion
and Bugzilla Bug 34206
before():execution(new(..)) does not throw NoAspectBoundException
All exceptions that occur during the static intialization of a persingleton
aspect will be swallowed. When using that aspect (via aspectOf())
a NoAspectBoundException will be thrown with the original exception
from the staitc initializer as the cause.
|
| |
|
|
| |
compiler issues error on inner aspects when privilieged
|
| |
|
|
|
|
|
|
| |
declaring interface methods should work as it does in interface
and Bugzilla Bug 45676
AspectJ enhanced code can not be used with plain old java anymo
and Bugzilla Bug 43972
Static crosscutting makes interfaces unusable for javac
|
| |
|
|
| |
After throwing advice on ctors doesn't execute for inter-type decls
|
| |
|
|
| |
default mode (inherited behaviour from JDT).
|
| |
|
|
| |
"declare soft" inside nested aspect does not work
|
| | |
|
| |
|
|
| |
convert Throwable obj to string and ajc aborts
|
| |
|
|
| |
- using extra source locations for recent test case (and retitling)
|
| |
|
|
|
|
|
| |
XLint warning for call PCD's using subtype of defining type
also added extraSourceLocations to IMessage+ for
message with multiple source lines
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
IncompatibleClassChangeError at runtime when compiling with -1.4 option
|
| | |
|
| |
|
|
|
|
|
| |
java.lang.VerifyError in generated code
Bug was that a portion of ReferencePointcut.concretize was not behaving
functionally but was mutating state.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Language regression, or possible language improvement?
The problem was caused by moving name binding in pointcut declarations to
happen before declare parents are evaluated. Because of this, the
compiler doesn't know that ContainerDescriptor isa Key when resolving
the ContainerLoader.containerLoads reference.
The change in ordering was made to fix a bug reported in declare error
and declare soft whose pcds where being evaluated before name binding
had happened in the pointcut declarations. Unfortunately, declare error
and declare soft are concretized at the same time as declare parents
(and all other declares ;-), so this move also led to the regression
noted above.
|
| |
|
|
|
|
|
|
|
|
|
| |
Bugzilla Bug 42740
declare error fails on pointcuts composed from multiple classes
Bugzilla Bug 42746
within() pcd is confused for certain declare softs
and probably:
Bugzilla Bug 42739
Compiler crash in ajc head (post 1.1.1 rc1)
|
| |
|
|
| |
Regression: Cannot compile complex weaving anymore
|
| |
|
|
|
|
|
|
|
|
|
| |
This bug is a reversion from 1.1.0 and so was a high-priority to get fixed
before the 1.1.1rc2 release tommorrow. The bug was caused by a
conflict between the implementation of perthis and a fix made in 1.1.1rc1
to the code for signature matching to handle inter-type declarations
correctly. The fix was the addition of two null pointer checks.
I hope that the original reporter will file a bug report in bugzilla tomorrow
for tracking.
|
| |
|
|
| |
throw derivative pointcuts not advised
|
| |
|
|
| |
percflow aspects compiled from jars share one instance for all entry points
|
| | |
|
| |
|
|
| |
Weaving failure when using injars
|
| |
|
|
|
|
|
|
|
|
| |
binary aspect-declared methods conflict, lost their exception clauses
larger fix to address more issues with aspect-declared methods in bytecode form
also addressed declared exception issue more generally
this fix should be more stable than a smaller hack that just fixed this specific bug
report would have been. added a few tests to match increased scope.
|
| |
|
|
|
|
| |
binary libraries
Corresponding non-binary passing tests added to ajcTests.xml
|
| |
|
|
| |
some should fail) -- confirming on the list.
|
| |
|
|
| |
call pointcut with interface type fails if method declared on interface
|
| |
|
|
|
|
| |
super-qualified pointcut reference cause weaver stack trace
and Bugzilla Bug 40814
no error when defining interface pointcuts
|
| |
|
|
|
|
| |
ClassFormatError on external subtype-qualified ref to supertype pointcut
This was an important bug that was caused by the static fields used in the implementation of cflow being placed on the wrong class. This broke the rules used for name mangling and could occasionally result in name collisions as shown here.
|
| |
|
|
| |
aspects
|
| |
|
|
| |
Class Literals as non final fields
|
| |
|
|
|
| |
convenience method that was removed. Updated tests to enable the
test for the horrible switch bug that this fixes.
|
| |
|
|
|
|
| |
Default method impl for interface causes internal exception.
test submitted by George Harley
|
| |
|
|
| |
Compiler crashes in jar and cflow
|
