Bug 107741 - Update WeavingURLClassLoader
Summary: Update WeavingURLClassLoader
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: Library (show other bugs)
Version: DEVELOPMENT   Edit
Hardware: PC Windows XP
: P2 enhancement (vote)
Target Milestone: 1.5.0RC1   Edit
Assignee: Matthew Webster CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 71066
  Show dependency tree
 
Reported: 2005-08-23 11:46 EDT by Sian January CLA
Modified: 2005-10-14 07:42 EDT (History)
0 users

See Also:

Attachments
Enhanced WeavingURLClassLoader (13.60 KB, application/octet-stream)
2005-10-13 08:43 EDT, Matthew Webster CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sian January CLA 2005-08-23 11:46:38 EDT 
The WeavingURlClassLoader needs to be updated to work with the most recent 
version of ApsectJ so that it can be used by AJDT to provide IDE support for 
load time weaving.
Comment 1 Adrian Colyer CLA 2005-08-26 11:56:45 EDT 
for M4
Comment 2 Adrian Colyer CLA 2005-09-27 05:54:23 EDT 
Raising to P2, need to get this into the code base very soon.
Comment 3 Matthew Webster CLA 2005-10-13 08:43:08 EDT 
Created attachment 28230 [details]
Enhanced WeavingURLClassLoader

Patches
loadtime:	       WeavingURLClassLoader (add)
org.aspectj.ajdt.core: AjcTestCase
testing:	       XMLBasedAjcTestCase
tests:		       LTW reweavable test
weaver: 	       WeavingURLClassLoader (remove)
Comment 4 Matthew Webster CLA 2005-10-13 08:43:28 EDT 
Changes
1. Moved WeavingURLClassLoader and accompanying test from weaver to loadtime 
project so that it can use ClassLoaderWeavingAdaptor. 
2. Changed WeavingURLClassLoader to use ClassLoaderWeavingAdaptor when no 
aspect URLs supplied, otherwise use WeavingAdaptor.
3. Enhanced XMLBasedAjcTestCase to support LTW using modified 
WeavingURLClassLoader. This is provide at least one use-case in the harness. 
Added a simple reweavable test

Patches
loadtime:              WeavingURLClassLoader (add)
org.aspectj.ajdt.core: AjcTestCase
testing:               XMLBasedAjcTestCase
tests:                 LTW reweavable test
weaver:                WeavingURLClassLoader (remove)
Comment 5 Andrew Clement CLA 2005-10-14 03:24:58 EDT 
Patch applied - only thing I changed was that although the
WeavingURLClassLoaderTest was moved across to a different module, it wasn't
added to the root test class for that module - I've now put it into
LoadtimeModuleTests.

waiting on build before closing.
Comment 6 Andrew Clement CLA 2005-10-14 07:42:32 EDT 
build available.