Fix for Bugzilla #39479, #40109
based on patch contributed by Andy Clement

Generalizes the patch with a method org.aspectj.weaver.bcel.Utility.copyInstruction
that works-around the bug in Select.copy().  Changed all calls to
Instruction.copy() to use this new method, would be nice to add the
rule:
     * declare error:
     *     call(* Instruction.copy()) && within(org.aspectj.weaver)
     *       && !withincode(* Utility.copyInstruction(Instruction)):
     *     "use Utility.copyInstruction to work-around bug in Select.copy()";

looks good.  fixed "org.eclipse.ajdt.core",
added how to build, jdt.core patch/sources

minor updates listed in the "recent changes" section.

surface ".." form for some number of arguments.
I'm using a different version of acrobat, so this might
cause problems.

added caveat for around advice on initialization

Updated modules documentation.

Added ASM documentation.

Updated dependency list.

Removed project dependancy on taskdefs project.

Updated module docs.

Finally adding some design docs (after 4 years).

Added safeguard for missing 1.3 setLocationRelativeTo method.

Fixed bugs introduced in 1.1rc2, where cast of message was incorrect.

test for Bugzilla Bug 39479
   NPE in bcel.LazyMethodGen when delegating from one ctor to a second that includes a switch.

added test for Bugzilla Bug 39462
   Compiler crashes in jar and cflow

tests and fixes for two awful error message bugs:
[Bug 39458] New: Compiler crash in ajc 1.1
[Bug 39460] New: Missing import crashes compiler

replacing 1.2 Vector API's with equivalent 1.1 API's to make telecom testable under 1.1

Updated post 1.1.0 release build.

Updated for 1.1.0 release

change only in comment echoed to output, listing expected failures for the link checker.

- linkcheck
- 1.1 examples (disabled pending hang fix)
- separate examples, since nonGui goes OOM under 1.3

adding keyword knownLimitation-run11

quick and dirty link checker manual test

quick and dirty link checker

link fixes

moved to docs

1.1 doc changes
- moving readme's to docs
- finishing readme-11 as change/porting
- misc...

1.1 updates

handler after/around advice limitation

added test for Bugzilla Bug 38345
   VerifyError, Inconsistent stack height with try/switch/if combination

updated for testing 1.1 classes,
(exclusive of changes to
 docs/dist/doc/examples/telecom/Call.java)

former version was built with -target 1.2, so it has a bad major version when running under 1.1

javarun attributes for interpreting error/output streams as errors,  used only when forking

corrected property name for forking run classpath

added output snooper to ProcessController for testing client

Expanded error message to make it clear that it is coming from parsing.

better unwrapping of exceptions

harness system.exit tests

adding examples, without testing from this script.  bad.

Added SecurityManager which halts any System.exit() call from JavaRun thread and puts the exit code in the status object.
This enables us to run the examples.

normal validation now fail if an Integer result not of value 0

tests harness' handling of code calling System.exit

test suite for example code in ../docs/dist/doc/examples

bootstrap jar for trim fixes

Trim testsrc but not testing dependencies for testing modules.

build of testing jars changed slightly.
Have to build non-all testing-drivers to get testing classes target.
Relies on updated Module.java which trims testsrc but not testing dependencies for testing modules.

- fixed keyword for ajctaskCompiler tests
- disabled ajdeCompiler tests as hanging (swing thread?)
- added pass-through variable for aspectj build properties
- now testing for Testing module tests,
  to force fix for testing modules not getting test classes

delay until after final last-modified-time stamp on newly-staged  files
since last-modified-time stamp can be delayed by filesystem buffering
and rounding-up to next second.

better info/debug strings when JavaRun fails
(also when status fails, not just when command fails)

better info/debug strings when JavaRun fails

delay until after final last-modified-time stamp on newly-staged  files
since last-modified-time stamp can be delayed by filesystem buffering.

Bugzilla Bug 38212
   can not resolve this member warning
This test currently passes, and therefore must be missing the bug; however,
this is the closest I could come to the bug as reported.

Script is usable!
- lots of rearranging and cleanup, docs
- now doing 1.3 and 1.4 testing
- more harness tests (some unverified)
- workaround OOM in examples
- cleanup after building-aspectj-with-aspectj

need more integration with manual process

needed two "expect-fail" keywords

no-op files to avoid CVS pruning empty directories

keywords knownLimitation-ajdeCompiler for correct but different results when running with -ajdeCompiler

permitting "none" as a "warn:" entry

- caveat for -nowarn per bug 38134
- documented -warn:none, since valid and used in -nowarn

test for Bugzilla Bug 38168
   insertion of lots of advice code can make branch offset for if too large
test entered as a knownLimitation for 1.1.0

slightly better error messages for Bugzilla Bug 38168
   insertion of lots of advice code can make branch offset for if too large

moved tests to ajcTests.xml as passing or knownLimitation

fix and test for Bugzilla Bug 38131
   ajc needs -d . option while correctly compiling classes from subpackage

neglected source

moved knownLimitations from ajcTestsFailing to ajcTests and labeled with keyword

Ant doc updates
- forkclasspath
- fork option classpath and conflicts
- added maven section

tests for devguide script example

more parts of tests for devguide script example

muddy caveat to JSR-45 comment

tests for setting message holder by class name

-Xlint:{level} tests

test checks that setting warn:constructorName works

fixing fork bugs:
- permitting fork in incremental mode only when using tag file
- pulling aspectjtools also from the task classpath
also failonerror default was not true as stated in the docs

addressing different results under 1.4

incorrect log file name for harness tests

better harness availability, control -- untested

PR-based queries were dominating - now do union correctly

documenting options

harness copyright wrong - was created in 2002

added keyword knownLimitations-ajctaskCompiler to the one command-line test that won't work with that wrapper.

- better logging, temp dir, deletion...
- using build.config system property

Fix for failure building with useEclipseCompiles testing-drivers initially (or any module that only indirectly depended on jdt.core, which has no sources)
- better logging
- checking zip results - returns false positive for empty zips
- compile(..) API change for adopting classes

Also, copyright clearly wrong - file created last fall.

Fix for failure building with useEclipseCompiles testing-drivers initially (or any module that only indirectly depended on jdt.core, which has no sources)
- better logging
- checking zip results - returns false positive for empty zips
- compile(..) API change for adopting classes

delete, close, and isEmpty

fixing 2 bugs disclosed by running ajcTests.xml with -ajctaskCompiler: (1) compliance set incorrectly (-1.3 instead of 1.4) and (2) warn:... form was ignored.
These would break only when running from a Javac compiler adapter and using <compilerarg>

better logging

fixing 2 bugs disclosed by running ajcTests.xml with -ajctaskCompiler: (1) compliance set incorrectly (-1.3 instead of 1.4) and (2) warn:... form was ignored.
These would break only when running from a Javac compiler adapter and using <compilerarg>

updating harness documentation for new features

tests for new feature to select tests by title

suite files now constrained to be readable to avoid confusion with -ajctestTitle.. referents.

adding harness feature to select tests based on their title (description), reading from a comma-delimited list or a file containing PASS|FAIL lines (from -traceTestsMin), optionally restricted to FAIL.

updating build.jar to bootstrap changes to permit ant.jar in a testing module

no real changes, just better docs and an (untested) "all" target.

-noExit to avoid stopping a wrapping ant script

moved a command-line-driven test of classpath variants here

Better BuildModule tests following ant.jar updates.
These now build with assemble-all off and on, and
tests were updated to check by running classes.

To support the new testing ICommand wrapper for AjcTask, permit ant.jar in a testing module (ant is omitted from non-testing modules because it is normally deployed separately in ant/lib/ant.jar).
This does not affect regular module or product builds.

documenting -ajdeCompiler and -ajctaskCompiler selectors

added taskdefs to classpath to support AjcTaskCompileCommand

Tests for AjcTaskCompileCommmand adapter

ICommand adapter for Ant taskdef is simple for the non-incremental case and ugly otherwise.

supporting -ajctaskCompiler option with AjcTaskCompileCommand