aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2009-06-21More work on the HandleData annotation. Constructor seems to work fine, ↵Reinier Zwitserloot
static constructor not so much.
2009-06-21trivialReinier Zwitserloot
2009-06-21Bug fix: using string literals in any lombok annotation value would blow up ↵Reinier Zwitserloot
that processor.
2009-06-21Due to a java bug, constants in enums don't work, so instead the default ↵Reinier Zwitserloot
access level for @Getter and @Setter have now just been hardcoded in GetterHandler and SetterHandler. Added ability to look up the Node object for any given AST object on Node itself, as you don't usually have the AST object. Added toString() method generating to @Data, and this required some fancy footwork in finding if we've already generated methods, and editing a generated method to fill in binding and type resolutions. HandleGetter and HandleSetter have been updated to use these features. Exceptions caused by lombok handlers show up in the eclipse error log, but now, if they are related to a CompilationUnit, also as a problem (error) on the CUD - those error log entries are easy to miss! Our ASTs can now be appended to. When you generate a new AST node, you should add it to the AST, obviously. Getter/Setter have been updated to use this.
2009-06-20Runs eclipse on a mac.Reinier Zwitserloot
2009-06-19Added initial support for the @Data annotation. Currently produces getters ↵Reinier Zwitserloot
and setters only, not yet a constructor, toString, hashCode, or equals. HandleGetter and HandleSetter have been updated to handle static (theoretic; you can't put annotations on static fields normally). You can now make AnnotationValue objects using just an annotationNode and a target type, as well as check if a given annotationNode is likely to represent a target annotation type. This is in Javac and Eclipse classes. HandleGetter and HandleSetter can now be asked to make a getter/setter, and will grab access level off of a Getter/Setter annotation, if present.
2009-06-19Added info on the addNetbeans branch.Reinier Zwitserloot
2009-06-19Finding lombok.jar would fail if you had any spaces in the path. Fixed that.Reinier Zwitserloot
2009-06-19Moved ClassLoaderWorkaround from 'java.lombok' to 'java.lombok.eclipse' as ↵Reinier Zwitserloot
its clearly eclipse-specific.
2009-06-19Live reloading of the eclipse parser classes is technically supported by the ↵Reinier Zwitserloot
agent though I never tested it. I found a bug while browsing this code. fixed it.
2009-06-19Renamed EclipseParserPatcher to EclipsePatcher, as it patches not just the ↵Reinier Zwitserloot
parser, but also the CompilationUnitDeclaration class so we can store our AST in it for caching purposes.
2009-06-19List of stuff we still need to do as well as research notes on how to get there.Reinier Zwitserloot
2009-06-18Created a fully working HandleSetter for eclipse, and refactored ↵Reinier Zwitserloot
HandleGetter a little mostly to stuff common code into PKG.
2009-06-18Expanded the AST printers to support a target PrintStream, and expanded the ↵Reinier Zwitserloot
@PrintAST annotation to let you supply an optional filename. Useful particularly for IDEs, which don't usually have a viewable console. Also renamed the printers to just 'Printer', as they are already inner classes of a specifically named type (JavacASTVisitor & co).
2009-06-17AnnotationHandlers can now return a boolean to set if they actually handled ↵Reinier Zwitserloot
the annotation or not (previously, the presumption was they always handled the annotation). This is very useful for PrintAST on eclipse, because before this change, you'd never see method contents (as the initial dietParse would come first). Now Eclipse PrintASTHandler will skip any non-full runs, and only print non-diet. It then returns true only if it printed.
2009-06-17Added 'does setX/getX already exist' check to javac's HandleGetter/Setter. ↵Reinier Zwitserloot
If they exist (with any returntype/paramlist), no getter/setter is generated, and instead a warning is added on the annotation.
2009-06-17Added the SetterHandler for javac. Also added a way to get the SymbolTable ↵Reinier Zwitserloot
on a JavacAST.Node, because you need it to e.g. access constant types like 'void'.
2009-06-17Made the printing of Statements for @PrintAST slightly more useful (if a lot ↵Reinier Zwitserloot
more verbose), and bumped the version number in honour of quite a bit of redesign these past few commits.
2009-06-17A useful annotation that prints the AST of any annotated element via the ↵Reinier Zwitserloot
XASTPrinters in each ASTVisitor interface.
2009-06-17Just renamed some parameters to more sensical names.Reinier Zwitserloot
2009-06-17A bugfix to the visitMethodArgument method, and printing improved by ↵Reinier Zwitserloot
printing the raw (instead of resolved) types for fields etc, because usually they aren't resolved yet.
2009-06-17Moved the traverse() from Eclipse/JavacAST to Eclipse/JavacAST.Node, so that ↵Reinier Zwitserloot
you can start your traversal at any point, not just from the top. Also a bugfix for endVisitStatement which passed the wrong node, and method arguments in Javac are no longer misfiled as local declarations.
2009-06-17No retention at all seems much smarter, because then it wouldn't be ↵Reinier Zwitserloot
neccessary to have the lombok.jar in the deployment path of whatever you write WITH lombok, which is a good idea. Though - rewriting to e.g. a Lombok.sneakyThrow() call would require lombok.jar anyway. We'll cross that bridge when we get there.
2009-06-17Renamed the Handler implementations.Reinier Zwitserloot
2009-06-17TrivialReinier Zwitserloot
2009-06-17Removed a debug print.Reinier Zwitserloot
2009-06-17Error reporting now works 99% properly - the right position is being marked ↵Reinier Zwitserloot
off. The only issue is that the count by javac is being misreported (it doesn't count lombok errors).
2009-06-17Warnings and errors on specific positions works, sort of. Still missing ↵Reinier Zwitserloot
something crucial to properly make javac generate the line + the ^^^^ indicators.
2009-06-17NullPointerExceptions were showing up in Eclipse, when a 2+ dimensional ↵Reinier Zwitserloot
array of Statements contains inner arrays that are null. Fixed that.
2009-06-17Added reading in (via spi) JavacASTVisitors and calling them, as well as a ↵Reinier Zwitserloot
mechanism to generate generic errors and warnings for javac.
2009-06-17None of the methods in this class were even being used, so deleted the whole ↵Reinier Zwitserloot
thing.
2009-06-17Turns out using instanceof checks to figure out if a LocalDeclaration is a ↵Reinier Zwitserloot
method argument or not (by instanceof checking if it's an Argument) is faulty, as e.g. the argument to a catch block is also an Argument object. Rewritten the visitChild method to be based on a switch on the Node's getKind(), just like JavacAST. This even looks nicer.
2009-06-17Massive refactors. This list isn't complete, but should give you an idea:Reinier Zwitserloot
A) many things in lombok.eclipse moved to lombok.core to enable reuse with lombok.javac. B) lombok.javac works now similarly to eclipse's model: We first make big ASTs that are bidirectionally traversable, then we walk through that for annotations. C) Instead of getting an annotation instance, you now get an object that is more flexible and can e.g. give you class values in an enum as a string instead of a Class object, which may fail if that class isn't on the classpath of lombok. D) sources to the internal sun classes for javac added to /contrib.
2009-06-16Moved the generation of the Trees object to init(), as it seems to be the ↵Reinier Zwitserloot
same on every round.
2009-06-16Removed debug prints.Reinier Zwitserloot
2009-06-16Implemented a lot of stuff for javac, but we ran into 2 major issues still ↵Reinier Zwitserloot
to be implemented: 1. The visit mode of a lombok handler (does not trigger off of annotations, instead sees every field, method, type, and statement), needs to be coded, 2. triggering off of annotations via APT's annotation handling system skips method-local classes. We'll need to recode this via an AST visitor like we need for issue #1 Other than that, triggering off of annotations works swimmingly!
2009-06-16See diffReinier Zwitserloot
2009-06-16Added ability to add visitor handlers.Reinier Zwitserloot
2009-06-15Propagated the fact that you get the Node object belonging to the ↵Reinier Zwitserloot
annotation, and not the field/type/local/method it goes with, all the way, so that you can easily generate a warning on an annotation in a handler.
2009-06-15Switched the level of the Node object you get during a visitAnnotationOnX ↵Reinier Zwitserloot
call from the Field/Type/Method/Local to the Annotation, so that you can interact with its handled flag.
2009-06-15Added annotations as definitive children of nodes in our custom AST, and ↵Reinier Zwitserloot
updated the visitor to call a separate visitAnnotationOnX method for annotated stuff. This way, 'handled' can be set per annotation. Also fixed a bug in AST generation that caused StackOverflowErrors on most source files, and did some cosmetic renaming of parameters.
2009-06-15trivialReinier Zwitserloot
2009-06-15all eclipse AST Statements objects are now part of the custom AST we build ↵Reinier Zwitserloot
for lombok. This way something like @AutoClose on a local var declaration can walk up one node, find all mentions of the variable, and add a close call right after the last mention.
2009-06-15Fixed showstopper bugs we ran into when actually running with these changes. ↵Reinier Zwitserloot
Related to 8353911b1d3a8d59a07042976bb924a7eccb5d0d
2009-06-15Just in case people run lombok.jar as a java app (e.g. by double-clicking ↵Reinier Zwitserloot
it), open a browser window with info, and print some useful help to the console.
2009-06-15Moved version.txt to lombok.core.Version.java.Reinier Zwitserloot
2009-06-15Renamed lombok.transformations lombok.core as the purpose of this package is ↵Reinier Zwitserloot
to contain stuff that is useful for any lombok implementation (be it e.g. javac via apt or eclipse via agent), but not annotations and other classes that are for 'end users'.
2009-06-15Moved TypeResolver to the eclipse package, as it has eclipse-specific code ↵Reinier Zwitserloot
in it.
2009-06-15Renamed lombok.agent.eclipse to lombok.eclipse.agentReinier Zwitserloot
2009-06-15Now you no longer need to add a -Xbootclasspath/a parameter to eclipse.ini; ↵Reinier Zwitserloot
the agent fixes the classpath all by its lonesome. Wahey!