Bug 166238 - -nowarn doesn't work for load-time weaving
Summary: -nowarn doesn't work for load-time weaving
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: ---   Edit
Assignee: aspectj inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-11-29 13:34 EST by Ron Bodkin CLA
Modified: 2012-04-03 15:32 EDT (History)
0 users

See Also:

Attachments
Fixes the problem (808 bytes, patch)
2006-11-29 13:35 EST, Ron Bodkin CLA 		aclement: iplog+
Details | Diff
Test cases. Patch to tests module. (3.02 KB, patch)
2006-11-29 13:38 EST, Ron Bodkin CLA 		aclement: iplog+
Details | Diff
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 Ron Bodkin CLA 2006-11-29 13:34:21 EST 
The -nowarn flag is passing through to not ignore warnings (I think the logic used to be convoluted here but now the underlying implementation does what it says, the CLWA has the wrong logic).

See attached patches for a test case and a fix.
Comment 1 Ron Bodkin CLA 2006-11-29 13:35:42 EST 
Created attachment 54730 [details]
Fixes the problem
Comment 2 Ron Bodkin CLA 2006-11-29 13:38:54 EST 
Created attachment 54731 [details]
Test cases. Patch to tests module.
Comment 3 Matthew Webster CLA 2006-12-01 07:15:26 EST 
There didn't appear to be an existing test for -nowarn. I have moved the tests to the new Ajc160Tests for consistency. Thanks for a great patch Ron.
Comment 4 Matthew Webster CLA 2006-12-01 08:23:31 EST 
Patches committed

iplog <50
Comment 5 Matthew Webster CLA 2006-12-01 10:43:55 EST 
fixe available.