bug 43033 - NPE in concretize error path for ReferencePointcut

fix and test for Bugzilla Bug 43194
   java.lang.VerifyError in generated code

Bug was that a portion of ReferencePointcut.concretize was not behaving
functionally but was mutating state.

make canonical versions of files...

test for canonical versions of sourcedirs etc.

always canonicalize the file argument when searching for nodes by file and/or line.

fix and test for Bugzilla Bug 42993
   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.

we fixed a few more bugs! (update total to 48)

added a cache by type to avoid linear searches

remember handleIdentifier once calculated, use canonical file path map

added a cache of canonical file paths

remember source location and handle identifier across invocations

fix and tests for at least 2 bugs:
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)

removing a debugging Thread.dumpStack() that made for confusing
error messages

added in building architecture for turoial exercises

Ant task revisions

added pr number for Bugzilla Bug 42652
   Regression: Cannot compile complex weaving anymore

updated with list of major fixes / enhancements in 1.1.1

@testcase PR#XXX omnibus privileged access from aspectj-users mail "Problem with interfaces in method's signature"

in case anyone wants them...
- a compile-speed script
- a script that uses Jikes to incrementally compile AspectJ

JoinPointCollector.java was moved to sandbox api-clients, and in any case has not been updated for the new asm API's.

updated tests - all passing

dirty example adapted from May user email

pointcuts fixed, so library updated/fixed and tested

pointcuts fixed, so library updated/fixed

initialization example: more comments and more testable

using linked list, more comments

@testcase PR#42668 after returning type incompatible with join point return type

sample api code to list affected files

Exposed file creation to allow overriding (for the use of canonicalization).

Added path canonicalization to config parser.

Removed asm path canonicalization.

Added model building performance test.

Added spacewar to test data.

Made model generation optional.

test and fix for bug reported today on aspectj-dev
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.

test and fix for Bugzilla Bug 42539
   throw derivative pointcuts not advised

aligning task with Javac task:
- copies resources by default when using the Javac adapter
- does not copy resources unless specified when invoked via AjcTask
- copies injar resources always

sourceRootCopyFilter test

changed warning to error

mostly final edits

fix and test for Bugzilla Bug 41359
   percflow aspects compiled from jars share one instance for all entry points

updated fix for 40807 - compiler adapter was supplying destDir when not specified
Also adding warning for resource-copying (36071; copyInJars, sourceRootCopyFilter)
I would like to disable entirely, but don't see and haven't gotten confirmation of behavior from last week's telephone call.

cloning re-used test specifications to avoid test interactions

ignoring test case expecting a skip when using the old ajc compiler in incremental mode,
since we're not using ajc 1.0 anymore.
Should revisit if we do.

checking clone when doing xml round trips on our test specifications

compiler run equals check wasn't testing argfiles, aspectpath, or sourceroots

implementing clone() for test specifications

serializing tests.  Some depended on others for builds, but test run order is not guaranteed.

updated section 3

UML diagram built with Argo UML (http://argouml.tigris.org)

made everything inherit from Test again
updated section 3 tests

moved to List

typo

in AspectJ 1.1, set jp's have void return type

fixes

removed unnecessary constructors and untabified

updating for new files

stub answer file

updated section 2

rough edit of text for section 3

revised section 4 (untested)

untabified

untabified

initial build.xml; has lots of dangling references

starting rework for oopsla

Fixed several broken links uncovered in 1.1.1rc1 testing.

performance faq item corrections promised on the list.
I need some actual facts for a real answer.

added a cache by handle to avoid terrible performance on larger projects

@testcase PR#41888 call PCD fails when given subtype of defining type

Using correct patch for bug 39436 instead of vague one from email

Still patching for bug 39470 (patch devised by Andy, vetted by me).
Sorting the local variables so we can deal with the fact that BCEL
generates extra local variables and doesn't bother to make them
hashable.

added ant driver for JUnit ajctest driver

initial cut at resolver.  Still getting default JDK 1.4 parser error "base URL required for relative references.

disabled test spec error test b/c it can't be skipped for JUnit  driver

Reimplemented JUnit harness adapter

balancing start/end suite calls

some xml parsers don't like space here...

paring globals to those used, then using them consistently

permitting null input in public utility method

Added a call to remove all structure model listeners from previous run to prevent
structure listener leakage.

Added a removeAllListeners method to be called from test suite to
prevent listener leakage which cripples performance of the test cases.

deleted a rogue import of com.sum.crypto... that mik must have added by accident.
I suspect this is what was breaking the build!

Fixed AJDT bug with truncation of labels (there was a weird substring call in AdviceActionDelegate). Fixed ASM/AJDT modifiers bug.  Fixed ASM static initializers bug (we may want to change the way they show up, i.e.   Added imports to ASM containment hierarchy.  Fixed numerous AJBrwoser, JBuilder and NetBeans bugs.

Added spacewar project for testing.

Added declare error/warning relationships to model.

Expesed "isError" field with getter (used by asm relationship provider).

copyright header change to satisfy BuildModuleTests

*** empty log message ***

Ported inline annotation support to new ASM API.  Ported aspect visualizer.  Improved link siganture generation.  Improved ASM test coverage.

wire in older minimal test cases

static methods for unwrapping to keep internal structure knowledge local

disabled tests

comment about disabled tests

Improved declare relationships.  Implemented batch-build relationship clearing policy.

Updated org.aspectj.asm relationship model to string-handle-based API in order to support adding and removing relationships at any point in the compilation cycle, and to support external tools building relationships (e.g. JDT's incremental containment hierarchy builder).  Also made inter-type declaration relationships show up in the model.

Added support for multiple relationships for a single program element.  Renamed model container class.

Minor improvements to structure model generation, clean up of test suite output, and port of AJDT to new ASM APIs.

Minor improvements to structure model generation, clean up of test suite output, and port of AJDT to new ASM APIs.

use System property aspectjrt.path to add to testcase classpath if defined

Expanded developer documentation.  Moved ASM doc to MSWord format since it will have graphics and revision support will help.