These changes are both (a) a performance optimization, and (b) an
improvement on the binding across || rules that we implemented in
1.2.1.
Instead of saying the the first binding in a left-to-right traversal of the
pointcuts DNF is the one that you get (too much to ask users to do DNF
rewriting in their heads), this version implements the rule that every
|| branch in the DNF must bind all formals, and if two || branches can have
any join points in common (can match join points of the same kind), then
both must bind all formals identically.

So it allows things like execution(* *(..)) && args(x) || call(* *(..)) && this(x)
which previously we used to forbid.

But primarily it turned out to be a performance optimization.

 	declare soft can cause programs with invalid exception behaviour to be generated

	  	around advice throws java.lang.VerifyError at runtime

	  	passing null to array arguments confuzes static join point signature.

	  	Bogus error message: The abstract method ajc$pointcut$$tracingScope$a2 in type Tracing can only be defined by an abstract class

	  	AJC possible bug with static nested classes

 	declare warning warns at wrong points

	  	The introduction on interface causes the interface implementation class error
(testcase now expects the error text)

	  	The introduction on interface causes the interface implementation class error

 	wrong line for method execution join point

 	Inconsistency in scoping of protected members in ITDs

 	ITD name clashes with private members

	  	proper handling of ExceptionInIntializer inside <clinit> in presence of after throwing advice

Bugzilla Bug 67578: Privileged Aspect Access Problem Across Packages
Bugzilla Bug 67579: NPE on privileged aspect error

	  	NoSuchMethodError calling private method from around advice in inner aspect

 	scope for intertype methods

	  	Compile time declarations (warning and error) do not accept string concatenation (with +)

	  	intertype initialisers should match field set pointcuts