Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Avoid ThreadLocal memory leak in JoinPointImpl | Alexander Kriegisch | 2024-03-12 | 1 | -29/+16 |
| | | | | | | | | | | | | | | | | | | | | | | | according to https://rules.sonarsource.com/java/tag/leak/RSPEC-5164/. Now, there no longer is a thread-local stack of AroundClosure instances, but rather a list of them, which can only grow but never shrink. Instead, we now have a thread-local (integer) list index, for every thread being initialised with pointing to the last element. I.e., every thread can unwind by decrementing the index while proceeding, independently of other threads. A positive side effect is that this approach also works for long-lived threads from thread pools, used by executor services. Hence, test Bugs199Tests.testAsyncProceedNestedAroundAdviceThreadPool_gh128, which was previously commented out, has been activated and passes, see #141. I am not sure if this brings @AspectJ style, non-inlined, nested around advice execution functionally on par with native ones, but at least for current scenarios it seems to work. Fixes #288, #141. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name> | ||||
* | Globally fix spelling error "dependancy/dependancies" | Alexander Kriegisch | 2024-01-06 | 2 | -2/+2 |
| | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name> | ||||
* | Remove redundant casts after generics update | Andrey Turbanov | 2022-04-12 | 7 | -10/+10 |
| | |||||
* | Make Stack<AroundClosure> in JoinPointImpl InheritableThreadLocal | Alexander Kriegisch | 2022-03-23 | 1 | -6/+19 |
| | | | | | | | | | | In case of asynchronous proceeding for nested around-advice, child threads need copies of the original around-closure stack. Otherwise, the target method thread will have popped the stack empty after unwinding. Fixes #128. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name> | ||||
* | Cleanup redundant type casts, due to too weak variable type declaration | Andrey Turbanov | 2022-02-26 | 1 | -2/+1 |
| | |||||
* | Update 'runtime' code to use generics | Andrey Turbanov | 2022-01-10 | 24 | -98/+98 |
| | |||||
* | Replace uses of StringBuffer with StringBuilder. | Andrey Turbanov | 2021-11-20 | 13 | -13/+13 |
| | | | | StringBuffer is a legacy synchronized class. StringBuilder is a direct replacement to StringBuffer which generally have better performance. | ||||
* | Upgrade license from CPLv1/EPLv1 to EPLv2 | Alexander Kriegisch | 2021-06-04 | 122 | -1284/+1284 |
| | | | | | | | This was required by the Eclipse team as one precondition for the next release. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name> | ||||
* | Remove unnecessary interface modifiers | Lars Grefer | 2020-08-17 | 11 | -286/+286 |
| | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de> | ||||
* | Cleanup unused imports | Lars Grefer | 2020-08-16 | 2 | -4/+0 |
| | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de> | ||||
* | Use the diamond operator where possible | Lars Grefer | 2020-08-13 | 3 | -27/+27 |
| | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de> | ||||
* | Unnecessary unboxing | Lars Grefer | 2020-08-08 | 3 | -8/+8 |
| | | | | | | Reports "unboxing", e.g. explicit unwrapping of wrapped primitive values. Unboxing is unnecessary under Java 5 and newer, and can be safely removed. Signed-off-by: Lars Grefer <eclipse@larsgrefer.de> | ||||
* | Unnecessary boxing | Lars Grefer | 2020-08-08 | 1 | -7/+7 |
| | | | | | | Reports explicit boxing, i.e. wrapping of primitive values in objects. Explicit manual boxing is unnecessary under Java 5 and newer, and can be safely removed. Signed-off-by: Lars Grefer <eclipse@larsgrefer.de> | ||||
* | 'String.indexOf()' expression is replaceable with 'contains()' | Lars Grefer | 2020-08-08 | 4 | -10/+10 |
| | | | | | | Reports any String.indexOf() expressions which can be replaced with a call to the String.contains() method available in Java 5 and newer. Signed-off-by: Lars Grefer <eclipse@larsgrefer.de> | ||||
* | 'for' loop replaceable with enhanced 'for' loop | Lars Grefer | 2020-08-08 | 4 | -10/+10 |
| | | | | | | Reports for loops which iterate over collections or arrays, and can be replaced with an enhanced for loop (i.e. the foreach iteration syntax). Signed-off-by: Lars Grefer <eclipse@larsgrefer.de> | ||||
* | Polish | Andy Clement | 2020-07-22 | 1 | -37/+37 |
| | |||||
* | polish | Andy Clement | 2020-04-20 | 1 | -0/+2 |
| | |||||
* | Include JDTCore for Java14 | Andy Clement | 2020-04-20 | 2 | -32/+35 |
| | |||||
* | 333274: more tests and fixes: nested @Around advice with proceed | Andy Clement | 2019-02-19 | 3 | -20/+82 |
| | |||||
* | polish javadoc in runtime module | Andy Clement | 2019-02-07 | 59 | -467/+324 |
| | |||||
* | Add module tests back in - but streamlined | Andy Clement | 2019-01-31 | 2 | -2/+43 |
| | |||||
* | mavenizing runtime module - merged in aspectj5rt | Andy Clement | 2019-01-24 | 71 | -0/+5713 |
| | |||||
* | mavenizing runtime module - complete | Andy Clement | 2019-01-24 | 53 | -1/+287 |
| | |||||
* | polish javadoc | Andy Clement | 2018-10-15 | 1 | -12/+12 |
| | |||||
* | Bug#531694: generate more optional thisJoinPoint construction code | Andy Clement | 2018-03-09 | 1 | -9/+116 |
| | | | | | | | This commit introduces some new methods into the runtime Factory class and modifies code generation to use them (and to use the form of the LDC bytecode that loads class constants). | ||||
* | Fix 500035: handling target only binding in @AJ pointcut | Andy Clement | 2016-11-18 | 1 | -3/+11 |
| | |||||
* | Cleaning up thread locals when counters/stacks hit zero | Andy Clement | 2013-03-22 | 7 | -1/+25 |
| | |||||
* | 323438 | aclement | 2010-08-23 | 1 | -0/+31 |
| | |||||
* | 89009: added getId() to joinpoint.staticpart | aclement | 2009-03-24 | 3 | -498/+577 |
| | |||||
* | 154427: test and fix | aclement | 2008-12-01 | 1 | -37/+81 |
| | |||||
* | 210848: better javadoc | aclement | 2008-02-21 | 1 | -2/+15 |
| | |||||
* | javadoc only - null Constructor for interface and static initializers, empty ↵ | wisberg | 2006-09-21 | 1 | -2/+13 |
| | | | | CodeSignatures values for static initializers. | ||||
* | Bug 155972 "NullPointerException in ConstructorSignature.getConstructor()" | mwebster | 2006-09-04 | 3 | -3/+3 |
| | |||||
* | 126167: Fix for @Around problems... | aclement | 2006-06-23 | 2 | -7/+82 |
| | |||||
* | test and fix for 145086 | aclement | 2006-06-03 | 1 | -12/+24 |
| | |||||
* | update license to EPL | acolyer | 2006-06-01 | 44 | -84/+84 |
| | |||||
* | tests and fix for 122253 | aclement | 2006-05-30 | 1 | -0/+188 |
| | |||||
* | fix for NPE in SignatureImpl reported on the list by Simon Walter | aclement | 2006-05-30 | 1 | -1/+5 |
| | |||||
* | synchronization joinpoints: aspectjrt changes | aclement | 2006-05-24 | 6 | -0/+154 |
| | |||||
* | fixes for 135780: aspectjrt will work better now on a 1.1 vm | aclement | 2006-05-03 | 1 | -12/+44 |
| | |||||
* | spelling fixed! (pr122417) | aclement | 2006-01-10 | 1 | -1/+1 |
| | |||||
* | moved to aspectj5rt | acolyer | 2005-12-19 | 1 | -188/+0 |
| | |||||
* | organized imports | aclement | 2005-11-10 | 1 | -2/+0 |
| | |||||
* | fix for pr104220 - poor toString implementations for adviceexecution join ↵ | acolyer | 2005-11-08 | 3 | -5/+19 |
| | | | | points. | ||||
* | fix for pr59076, much more memory efficient creation of join point static parts | acolyer | 2005-09-01 | 2 | -38/+159 |
| | |||||
* | Fix and tests for PR94167: NPE in reflection API. Fix submitted by Ron Bodkin. | aclement | 2005-06-07 | 2 | -2/+2 |
| | |||||
* | hacked out 1.4 apis - needs tidying up but I'm just trying to see if I can ↵ | aclement | 2005-05-04 | 1 | -3/+6 |
| | | | | get the build to work. | ||||
* | PTW perClause for @AJ + perClause test | avasseur | 2005-05-02 | 1 | -10/+102 |
| | |||||
* | @AJ annotation are runtime visible, fixed all licenses from branch for EPL | avasseur | 2005-04-22 | 2 | -12/+14 |
| | |||||
* | fixing a missing "throw" clause | avasseur | 2005-04-22 | 1 | -1/+1 |
| |