tidy up after yourself

Bugette: annotation value of primitive type 'char' wasnt working !

optimization: rather than continuously constructing new arrays from the linked lists of instructions, we create one array and use it multiple times in MethodGen.

optimization: less fiddling with collections when setting up the localvariables for a woven method.

optimization: use bit twiddling rather than Sets for manipulating the list of Pointcut.couldMatchKinds - (saves a few percent when building shadows in my tests...)

optimization: char rather than string manipulation

optimization: dont unnecessarily build signatures and faster signature processing.

organized imports

lessen the cost of hashcode and equals

fix for 119882 (from Helen) - similar to fix for 113531

Progress on: empty catch block warning - compiler changes to warn on empty catch block

Progress on: empty catch block warning - option support

Progress on: empty catch block warning - test cases

Progress on: empty catch block warning - updates to test cases with empty catch blocks

Progress on: empty catch block warning - extra lint message

define some constants to avoid spreading string literals around

provide a way to reset dump state to free up memory

free up memory anchored in static at the end of a run

I don't think this class is actively used anymore...

don't hold onto references to AjBuildConfigs

don't hold eclipse source types in ajstate

go "back to the source" on an incremental aspect change

Put compilation-and-weaving-context into multi-thread mode when doing LTW

don't hold hard references to state in this context. Assume single threaded unless told otherwise. Provide reset to ditch memory we are holding.

comment fix

fix for 120356 - better feedback for DEOW when using @AJ. (from helen)

fix for 123212 - ignoring subversion resources

lazy unpacking of the line number table contents.  Only done when required - usually when a MethodGen built for a Method (i.e. we are going to weave it).

lazy unpacking of the table contents.  Only done when required - usually when a MethodGen built for a Method (i.e. we are going to weave it).

dont do anything if we dont have to.

not perfect yet - but some I'm using.

Helen and Eduardos AST changes for 110465

Progress on: Bug 126328: IlegalStateException : zip file closed
https://bugs.eclipse.org/bugs/show_bug.cgi?id=126328

closes the loop hole with >1000 jars, and adds some extra sanity checks on potentially closed zip files (tested that the call is cheap) to give better error messages.

Progress on: Bug 126316: Invalid method signature: Pjava/lang/Enum<Ljava/lang/Object;>;
https://bugs.eclipse.org/bugs/show_bug.cgi?id=126316

  --> fix and test case

Progress on: Bug 126316: Invalid method signature: Pjava/lang/Enum<Ljava/lang/Object;>;
https://bugs.eclipse.org/bugs/show_bug.cgi?id=126316
  --> test case to reproduce

more detail in memory answers, on incremental mode and strong static reference to singleton aspects

simple visitor added that lets you discover if a pointcut binds this or target

Err, corrected to visit the right branch as well as the left...

@AJ around tests for 126167

thats better.

oops

fix for 125810 (from helen)

testcode for 125699 and 125810 from matthew. test for 125699 commented out.

free up maps during cleanup() processing

get build configuration via accessor, not direct field access

made all the state in AjState private so we can get a better handle on what's going on. Added accessors where needed and extensively commented the lifecycle of the key state we hold.

cleanup the EclipseFactory afterCompilation

added a cleanup method to release memory held in type maps

dom changes for 125809 (from helen)

support for -messageHolder command-line option. This is needed by the forking iajc ant task to support use of a custom message handler when forking.

support for -messageHolder : do nothing since this (obviously) has to be processed very early in ajc.Main

support for messageHolder attribute even when forking (see corresponding change in ajc.Main)

enh 123423 (expose PTW type) - code currently commented out until return type of getWithinType() decided...

fixes for 125475/125480 - matches nothing @AJ support.  Definetly *NOT* as clean as it could be, but works.

avoid NPE (125657)

testcode for 125475/125480 (from matthew) and for enh 123423 (expose PTW type) - last one currently commented out.

testcode for 125475/125480 (from matthew) and for enh 123423 (expose PTW type)

useful definition of STRING

fix and test for 123901.  test for 125475 (commented out)

fixes for the AST - see comment #26 in 110465.  Now exposes perclause/privileged information.

fix for 121395 - error message on stupid aspectpath

fix for 125405 - demote raws to simple on incremental compiles when necessary.

testcode for 125405

helens changes for 125295 - should keep AJDT happy.

includes the AST changes already checked into shadows - see pr110465

New tests for all the AST changes - and updated tests due to the AST being more comprehensive.

necessary magic.  Factory 'mechanism' used so that the JDT project doesnt have a direct dependency on the ajdt.core project. It references the factory name through a string and dynamically loads the type.

Subclass for the AST that is able to create pointcuts/declares/etc.  Similar ctors to the supertype and one method copied into this class that was creating the AST directly when it needs to use the factory now to ensure it creates an instance of this subtype.

new placeholder nodes for type patterns and signature patterns.  Before this change this info was being 'discarded' and not included in the constructed AST - now it is at least created and held onto.  Future work would create the various subnodes for the different kinds of typepatterns and signaturepatterns - if users want to start analysing them below their 'top level'

new distinct nodes for the different types of declare - all follow the standard ASTNode pattern of having properties defined (modifiers/javadoc/etc).

oops, crap comment earlier - fix for 125101

fix is checked in.

fix for 122452 - pattern parser for a duff pointcut definition.  Also fix for 125027 (deow length set wrong during parsing). latter fix from Helen.

test and fix for 125080 - mixing numbers of types in a generic type hierarchy

test and fix for 125080 - mixing numbers of types in a generic type hierarchy

testcode for 122452

test and fix for 124999

testcode for 124399

test and fix for 124654: generic annotation matching..

test and fix for 124808

test and fix for pr124803 - mixing numbers of tvars/tparams in a hierarchy

incremental and memory answers gleaned from matthew and andy

build the model.

profiles of the 1.5.0 release of AspectJ building spring 1.2.6

ant script for running profiling tests against aspectj with ajc source compile, binary weaving, load-time weaving, and AJDT simulation. Can configure target application and source of compiler (workspace or pre-built jars).

support for <jvmarg>...</jvmarg> when forking (needed by profiling scripts)

77166 - newarray joinpoint support. "-Xjoinpoints:arrayconstruction"

77166 - newarray joinpoint support. "-Xjoinpoints:arrayconstruction"

without this check, we sometimes hide earlier problems.

77166 - newarray joinpoint support. "-Xjoinpoints:arrayconstruction"

77166 - newarray joinpoint support. "-Xjoinpoints:arrayconstruction"

77166 - newarray joinpoint - testcode

122347 - at least evict *something* from the type map.  use softrefs but tailorable. also has some basic profiling support for deactivated right now.

unused import.

an additional interface that custom message handlers may choose to implement. Implementors get callbacks when a build is starting and when a build has finished (allowing them to perform before and after build processing, like closing output files etc..).

Use ant's TaskLogger for all output from our task, rather than writing directly to sys out / err. This commit also fixes the issue that running ant -verbose did not produce verbose output from iajc (it does now). I ran into this with a client yesterday, and it had me stumped for a while!

test and fix for 123612 - remember to clear the caches for declared annotations between compiles.

doc updates for 76101 from Helen - how to build AJ and stuff  it into AJDT.

tests for 123695 and 124105

fix for 123695 - allow for null (no annotations)