Bug 248814 - Modify ltw dump directory based on loader/weaver instance
Summary: Modify ltw dump directory based on loader/weaver instance
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: LTWeaving (show other bugs)
Version: DEVELOPMENT   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 1.6.2   Edit
Assignee: aspectj inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-26 20:24 EDT by Andrew Clement CLA
Modified: 2008-09-27 13:44 EDT (History)
0 users

See Also:

Attachments
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 Andrew Clement CLA 2008-09-26 20:24:10 EDT 
Always dumping to _ajdump is not helpful in a multi-threaded environment - it should include some weaver or classloader instance related material in the dump location name to prevent clashing.
Comment 1 Andrew Clement CLA 2008-09-27 13:44:55 EDT 
New option on dump allows per weaver dump directories.

<dump perclassloaderdumpdir="true"/>

The weaving context can define names for classloaders, these are included in the verbose output (see the prefix in [...] here):

[AppClassLoader@1f12c4e] info register classloader sun.misc.Launcher$AppClassLoader@1f12c4e

These names are also used for the sub directory created in _ajdump.

_ajdump/AppClassLoader@1f12c4e

from the example above.

mustdoc.