- Jaxx-commits - mailman.cloud.codelutin.com

r1923 - trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 20:09:18 +0200 (Tue, 18 May 2010) New Revision: 1923 Url: http://nuiton.org/repositories/revision/jaxx/1923 Log: remove nasty binding Modified: trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing/MemoryStatusWidget.jaxx Modified: trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing/MemoryStatusWidget.jaxx =================================================================== --- trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing/MemoryStatusWidget.jaxx 2010-05-18 18:08:14 UTC (rev 1922) +++ trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing/MemoryStatusWidget.jaxx 2010-05-18 18:09:18 UTC (rev 1923) @@ -26,8 +26,7 @@ <JComponent implements='java.awt.event.ActionListener' foreground='{Color.BLACK}' - background='{Color.WHITE}' - font='{new JLabel().getFont()}'> + background='{Color.WHITE}'> <javax.swing.Timer id='timer' javaBean='null'/> @@ -105,7 +104,7 @@ void $afterCompleteSetup() { - //FontRenderContext frc = new FontRenderContext(null, false, false); + setFont(new JLabel().getFont()); Rectangle2D bounds = getFont().getStringBounds(memoryTestStr, frc); Dimension dim = new Dimension((int) bounds.getWidth(), (int) bounds .getHeight()); setPreferredSize(dim);

1 0

r1922 - in trunk/jaxx-compiler/src/main/java/jaxx/compiler: . binding decorators tasks
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 20:08:14 +0200 (Tue, 18 May 2010) New Revision: 1922 Url: http://nuiton.org/repositories/revision/jaxx/1922 Log: - Evolution #625: Improve data binding with inheritance - Improve JAXXEngine and JAXXTask (simplify a lot...) Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXCompiler.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXEngine.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataBinding.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataSource.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/decorators/DefaultCompiledObjectDecorator.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/CompileFirstPassTask.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/CompileSecondPassTask.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/FinalizeTask.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/GenerateTask.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/InitTask.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/JAXXEngineTask.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/ProfileTask.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/StyleSheetTask.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXCompiler.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXCompiler.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXCompiler.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -110,7 +110,7 @@ * inner classes (including enumerations), because currently they don't * always resolve (but will generally run without error anyway). */ - public static boolean STRICT_CHECKS = false; + public static boolean STRICT_CHECKS; public static final String JAXX_NAMESPACE = @@ -1307,34 +1307,30 @@ return; } - // register the jaxx file into the engine - URL jaxxURL = getClassLoader().getResource( - className.replace('.', '/') + ".jaxx"); - URL classURL = getClassLoader().getResource( - className.replace('.', '/') + ".class"); - if (jaxxURL == null || classURL == null) { + // register the jaxx file into the engine - // unknown class - //FIXME-TC20100502 : should throw something or at least warn + URL jaxxURL = ClassDescriptorHelper.getURL(getClassLoader(), className, "jaxx"); + + if (jaxxURL == null) { + + // not a jaxx file, can quit now return; } - try { - File jaxxFile = URLtoFile(jaxxURL); - File classFile = URLtoFile(classURL); - if (classFile.lastModified() > jaxxFile.lastModified()) { - return; // class file is newer, no need to recompile - } - } catch (Exception e) { - // do nothing + if (!engine.isCompileFirstPassTask()) { + throw new IllegalStateException( + "Internal error: adding dependency class " + + className + " during second compilation pass"); } + //FIXME tchemit 20100518 Must unify this with ClassDescriptorHelper to use the less costless resource if (jaxxURL.toString().startsWith("file:")) { File jaxxFile = URLtoFile(jaxxURL); try { jaxxFile = jaxxFile.getCanonicalFile(); } catch (IOException ex) { - // ignore ? + // ignore ? (at least log it...) + log.error(ex); } String jaxxFileName = className.substring(className.lastIndexOf(".") + 1) + @@ -1343,12 +1339,7 @@ assert jaxxFile.getName().equalsIgnoreCase(jaxxFileName) : "expecting file name to match " + className + ", but found " + jaxxFile.getName(); - if (jaxxFile.getName().equals(jaxxFileName)) { // check case match - if (!engine.isFirstCompilePassTask()) { - throw new AssertionError( - "Internal error: adding dependency class " + - className + " during second compilation pass"); - } + if (jaxxFile.getName().equals(jaxxFileName)) { engine.addFileToCompile(jaxxFile, className); } } Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXEngine.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXEngine.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXEngine.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -60,10 +60,10 @@ /** configuration of the launchor and underlines compilers */ protected final CompilerConfiguration configuration; - /** original list of files to run */ + /** original list of files to compile (says the detected modfied files) */ protected final JAXXCompilerFile[] incomingFiles; - /** Files to be treated while compilation. */ + /** Files to compile */ protected final List<JAXXCompilerFile> compilingFiles; /** Warnings detected while running. */ @@ -103,7 +103,9 @@ addFileToCompile(compilerFile); } - incomingFiles = getCompilingFiles(); + // fix once for all incoming files to compile + incomingFiles = compilingFiles.toArray( + new JAXXCompilerFile[compilingFiles.size()]); } public JAXXEngineTask[] getTasks() { @@ -151,13 +153,34 @@ return incomingFiles; } - public JAXXCompilerFile[] getCompilingFiles() { - return compilingFiles.toArray( - new JAXXCompilerFile[compilingFiles.size()]); +// public JAXXCompilerFile[] getCompilingFiles() { +// return compilingFiles.toArray( +// new JAXXCompilerFile[compilingFiles.size()]); +// } + + public JAXXCompilerFile[] getFilesToCompile() { + List<JAXXCompilerFile> files = new ArrayList<JAXXCompilerFile>(); + for (JAXXCompilerFile file : compilingFiles) { + if (file.getCompiler() == null) { + files.add(file); + } + } + return files.toArray(new JAXXCompilerFile[files.size()]); } + public JAXXCompilerFile[] getCompiledFiles() { + List<JAXXCompilerFile> files = new ArrayList<JAXXCompilerFile>(); + for (JAXXCompilerFile file : compilingFiles) { + if (file.getCompiler() != null) { + files.add(file); + } + } + // always send a copy to be safe. + return files.toArray(new JAXXCompilerFile[files.size()]); + } + public boolean containsJaxxFileClassName(String className) { - for (JAXXCompilerFile file : getCompilingFiles()) { + for (JAXXCompilerFile file : compilingFiles) { if (className.equals(file.getClassName())) { return true; } @@ -165,7 +188,7 @@ return false; } - public boolean isFirstCompilePassTask() { + public boolean isCompileFirstPassTask() { return currentTask != null && CompileFirstPassTask.TASK_NAME.equals(currentTask.getName()); } @@ -362,8 +385,8 @@ */ public void addFileToCompile(File jaxxFile, String jaxxClassName) { - if (isVerbose()) { - log.info("file = " + jaxxFile + ", fqn = " + jaxxClassName); + if (log.isDebugEnabled()) { + log.debug("file = " + jaxxFile + ", fqn = " + jaxxClassName); } JAXXCompilerFile file = new JAXXCompilerFile(jaxxFile, jaxxClassName); Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataBinding.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataBinding.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataBinding.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -25,6 +25,7 @@ package jaxx.compiler.binding; +import jaxx.compiler.CompiledObject; import jaxx.compiler.CompilerException; import jaxx.compiler.JAXXCompiler; import jaxx.compiler.java.JavaFileGenerator; @@ -37,6 +38,7 @@ import java.util.ArrayList; import java.util.List; +import java.util.Set; /** * Represents a data binding in a JAXX file. <code>DataBinding</code> uses @@ -45,56 +47,43 @@ */ public class DataBinding { - /** - * Logger - */ + /** Logger */ protected static final Log log = LogFactory.getLog(DataBinding.class); - /** - * Id of the data binding - */ + + /** Id of the data binding */ private String id; - /** - * Real Id of the data binding (can be suffix by a number for css binding to avoid collisions) - */ + + /** Real Id of the data binding (can be suffix by a number for css binding to avoid collisions) */ private String realId; - /** - * Constant id build from the {@link #realId} and used instead of {@link #realId} in generated code - */ + + /** Constant id build from the {@link #realId} and used instead of {@link #realId} in generated code */ protected String constantId; - /** - * source of the data binding - */ + + /** source of the data binding */ private String source; + /** * A Java snippet which will cause the destination property to be updated with the current value of * the binding. */ private String assignment; - /** - * A internal flag to - */ + + /** A internal flag to */ private boolean quickNoDependencies; - /** - * Compiled data source - */ + /** Compiled data source */ protected DataSource dataSource; - /** - * code to add to processDataBinding (null if no binding) - */ + /** code to add to processDataBinding (null if no binding) */ protected String processDataBinding; - /** - * code to register the databinding (null if no binding) - */ + + /** code to register the databinding (null if no binding) */ protected String initDataBinding; - /** - * Extra method to add to the binding - */ + + /** Extra method to add to the binding */ protected final List<JavaMethod> methods = new ArrayList<JavaMethod>(); - /** - * internal state passed to {@code true} when {@link #compile(jaxx.compiler.JAXXCompiler)} method is invoked - */ + + /** internal state passed to {@code true} when {@link #compile(JAXXCompiler)} method is invoked */ private boolean compiled; /** @@ -180,7 +169,7 @@ * * @param compiler compiler which includes the data binding * @return {@code true} if the expression has dependencies, {@code false} otherwise - * @throws jaxx.compiler.CompilerException if a compilation error occurs + * @throws CompilerException if a compilation error occurs */ public boolean compile(JAXXCompiler compiler) throws CompilerException { @@ -212,7 +201,7 @@ // was compiled compiled = true; - if (DataBindingHelper.SHOW_LOG || log.isDebugEnabled()) { + if (dataSource.showLog()) { if (binding) { log.info("detect a databinding : " + this); } else { @@ -260,10 +249,27 @@ if (needTest) { buffer.append("if (").append(objectCode).append(") {").append(eol); } - buffer.append(JavaFileGenerator.indent(getAssignment(), needTest ? 4 : 0, false, eol)); + String assiggment = getAssignment(compiler, dataSource.getOverrideIds()); + buffer.append(JavaFileGenerator.indent(assiggment, needTest ? 4 : 0, false, eol)); if (needTest) { buffer.append(eol).append("}"); } return buffer.toString(); } + + protected String getAssignment(JAXXCompiler compiler, Set<String> overrides) { + String s = getAssignment(); + + if (overrides != null && !overrides.isEmpty()) { + String tmp = s; + for (String override : overrides) { + CompiledObject o = compiler.getCompiledObject(override); + tmp = tmp.replaceFirst(override + ".", o.getJavaCode() + "."); + } + log.info("Assignment with overrides (" + s + ") = " + tmp); + s = tmp; + } + + return s; + } } \ No newline at end of file Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataSource.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataSource.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataSource.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -52,7 +52,14 @@ import java.beans.Introspector; import java.io.StringReader; import java.lang.reflect.Modifier; -import java.util.*; +import java.util.ArrayList; +import java.util.Arrays; +import java.util.HashSet; +import java.util.Iterator; +import java.util.LinkedHashMap; +import java.util.List; +import java.util.Map; +import java.util.Set; /** * Represents a Java expression which fires a <code>PropertyChangeEvent</code> when it can be @@ -62,40 +69,29 @@ */ public class DataSource { - /** - * Logger - */ + /** Logger */ protected static final Log log = LogFactory.getLog(DataSource.class); - /** - * type attached to "null" constants in parsed expressions - */ + /** type attached to "null" constants in parsed expressions */ private class NULL { } - /** - * id of data source - */ + /** id of data source */ private final String id; - /** - * Constant id - */ + + /** Constant id */ protected final String constantId; - /** - * The Java source code for the expression. - */ + + /** The Java source code for the expression. */ private final String source; - /** - * The current <code>JAXXCompiler</code>. - */ + + /** The current <code>JAXXCompiler</code>. */ private final JAXXCompiler compiler; - /** - * List of detected tracker (if none found, it is not a data binding) - */ + + /** List of detected tracker (if none found, it is not a data binding) */ private final List<DataListener> trackers; - /** - * the delegate of property to be required - */ + + /** the delegate of property to be required */ private String objectCode; protected final List<JavaMethod> methods; @@ -116,7 +112,7 @@ this.source = source; this.compiler = compiler; this.methods = methods; - this.trackers = new ArrayList<DataListener>(); + trackers = new ArrayList<DataListener>(); } public String getObjectCode() { @@ -142,6 +138,11 @@ return b.toString(); } + public boolean showLog() { +// return DataBindingHelper.SHOW_LOG || log.isDebugEnabled(); + return DataBindingHelper.SHOW_LOG; + } + /** * Compiles the data source expression and listener. This method calls methods in <code>JAXXCompiler</code> * to add the Java code that performs the data source setup. Adding listeners to <code>DataSource</code> is @@ -150,13 +151,12 @@ * expression will be compiled and evaluated at runtime to yield the <code>DataSource's</code> listener. * * @return <code>true</code> if the expression has dependencies, <code>false</code> otherwise - * @throws jaxx.compiler.CompilerException - * if a compilation error occurs + * @throws CompilerException if a compilation error occurs * @throws IllegalStateException if data source was already compiled */ protected boolean compile() throws CompilerException, IllegalStateException { - if (DataBindingHelper.SHOW_LOG || log.isDebugEnabled()) { + if (showLog()) { log.info("======= Start compile of " + source); } JavaParser p = new JavaParser(new StringReader(source)); @@ -185,7 +185,7 @@ // scan accepted expressions to detect dependencies and track listeners for (SimpleNode node : expressions.keySet()) { - if (DataBindingHelper.SHOW_LOG || log.isDebugEnabled()) { + if (showLog()) { log.info("Will parse expression " + node.getText()); } scanNode(node); @@ -204,7 +204,7 @@ // no requirements - this.objectCode = ""; + objectCode = ""; } else { // build the fully test from requirements @@ -212,11 +212,16 @@ StringBuilder buffer = new StringBuilder(); Iterator<String> itr = result.iterator(); - buffer.append(itr.next()).append(" != null"); + + overrideIds = new HashSet<String>(); + + String realSource = getJavaCode(itr.next()); + buffer.append(realSource).append(" != null"); while (itr.hasNext()) { - buffer.append(" && ").append(itr.next()).append(" != null"); + realSource = getJavaCode(itr.next()); + buffer.append(" && ").append(realSource).append(" != null"); } - this.objectCode = buffer.toString().trim(); + objectCode = buffer.toString().trim(); } } castsExpressions.clear(); @@ -224,16 +229,34 @@ return isBinding; } + private Set<String> overrideIds; + + private String getJavaCode(String s) { + CompiledObject o = compiler.getCompiledObject(s); + if (o != null && o.isOverride()) { + + if (showLog()) { + log.info("Use an override identifier : " + o.getJavaCode()); + } + + overrideIds.add(s); + } + return s; + } + + public Set<String> getOverrideIds() { + return overrideIds; + } + /** * Examines a node to identify any dependencies it contains. * * @param node node to scan - * @throws jaxx.compiler.CompilerException - * ? + * @throws CompilerException ? */ private void scanNode(SimpleNode node) throws CompilerException { if (node.getId() == JavaParserTreeConstants.JJTMETHODDECLARATION || - node.getId() == JavaParserTreeConstants.JJTFIELDDECLARATION) { + node.getId() == JavaParserTreeConstants.JJTFIELDDECLARATION) { return; } if (log.isTraceEnabled()) { @@ -254,7 +277,7 @@ break; case JavaParserTreeConstants.JJTPRIMARYEXPRESSION: type = determineExpressionType(node); - if (log.isDebugEnabled()) { + if (showLog()) { log.debug("result of determineExpressionType for " + node.getText() + " = " + type); } break; @@ -276,11 +299,12 @@ * @return the class descriptor of the return type or null */ private ClassDescriptor determineExpressionType(SimpleNode expression) { - if (log.isTraceEnabled()) { - log.trace("for expression " + expression.getText()); - } assert expression.getId() == JavaParserTreeConstants.JJTPRIMARYEXPRESSION; SimpleNode prefix = expression.getChild(0); + if (log.isDebugEnabled()) { + log.debug("for expression " + expression.getText() + " - prefix " + prefix + " - nb childrens of prefix: " + prefix.jjtGetNumChildren() + ", nb childrens of expression : " + expression.jjtGetNumChildren()); + } + if (prefix.jjtGetNumChildren() == 1) { int type = prefix.getChild(0).getId(); if (type == JavaParserTreeConstants.JJTLITERAL || type == JavaParserTreeConstants.JJTEXPRESSION) { @@ -340,6 +364,12 @@ } return method.getReturnType(); } catch (NoSuchMethodException e) { + if (showLog()) { + log.info("Could not find method " + methodName + ", code : " + code + " on : " + contextClass); + for (MethodDescriptor descriptor : contextClass.getMethodDescriptors()) { + log.info(" - " + Modifier.toString(descriptor.getModifiers()) + " " + descriptor.getName() + "(...) : " + descriptor.getReturnType()); + } + } // happens for methods defined in the current JAXX file via scripts String propertyName = null; if (methodName.startsWith("is")) { @@ -360,15 +390,16 @@ // check this is a javaBean CompiledObject compiledObject = compiler.getObjects().get(propertyName); if (compiledObject != null && compiledObject.isJavaBean()) { - if (DataBindingHelper.SHOW_LOG || log.isDebugEnabled()) { - log.info("Detect a dependency from a script method '" + newMethod.getName() + "' which reflect a javaBean property " + propertyName); - log.info("Try to add a listener [symbol:" + bindingId + ",objectCode:" + "null" + "]"); + String objectCode = null; + if (showLog()) { + log.info("Detect a dependency from compiled object [" + objectCode + "]a script method '" + newMethod.getName() + "' which reflect a javaBean property " + propertyName); + log.info("Try to add a listener [symbol:" + bindingId + ",objectCode:" + objectCode + "]"); log.debug(">> lastnode = " + lastNode + "(), suffix = " + suffix.getText() + ", expression = " + expression.getText()); } addListener(bindingId, - null, - "addPropertyChangeListener(\"" + propertyName + "\", this);" + JAXXCompiler.getLineSeparator(), - "removePropertyChangeListener(\"" + propertyName + "\", this);" + JAXXCompiler.getLineSeparator()); + objectCode, + "addPropertyChangeListener(\"" + propertyName + "\", this);" + JAXXCompiler.getLineSeparator(), + "removePropertyChangeListener(\"" + propertyName + "\", this);" + JAXXCompiler.getLineSeparator()); } contextClass = newMethod.getReturnType(); } @@ -486,16 +517,16 @@ FieldDescriptor newField = compiler.getScriptField(memberName); if (newField != null) { contextClass = newField.getType(); - if (DataBindingHelper.SHOW_LOG || log.isDebugEnabled()) { + if (showLog()) { log.info("Detect a dependency from a script field '" + newField + "'"); log.info("Try to add a listenenr [symbol:" + tokensSeenSoFar.toString() + ",objectCode:" + null + "]"); } String eol = JAXXCompiler.getLineSeparator(); addListener(tokensSeenSoFar.toString(), - null, - "addPropertyChangeListener(\"" + memberName + "\", this);" + eol, + null, + "addPropertyChangeListener(\"" + memberName + "\", this);" + eol, // "addPropertyChangeListener(\"" + memberName + "\", " + listenerId + ");" + eol, - "removePropertyChangeListener(\"" + memberName + "\", this);" + eol); +"removePropertyChangeListener(\"" + memberName + "\", this);" + eol); // "removePropertyChangeListener(\"" + memberName + "\", " + listenerId + ");" + eol); assert contextClass != null : "script field '" + memberName + "' is defined, but has type null"; @@ -540,14 +571,14 @@ try { if (handler.isMemberBound(memberName)) { String bindingId = objectCode + "." + memberName + (method ? "()" : ""); - if (DataBindingHelper.SHOW_LOG || log.isDebugEnabled()) { + if (showLog()) { log.info("Detect a dependency from a event handler for memberName '" + memberName + "' for class " + objectClass); log.info("Try to add a listenenr [symbol:" + bindingId + ", objectCode:" + objectCode + "]"); } addListener(bindingId, - objectCode, - getAddMemberListenerCode(handler, objectCode, memberName, "this", compiler), - getRemoveMemberListenerCode(handler, objectCode, memberName, "this", compiler)); + objectCode, + getAddMemberListenerCode(handler, objectCode, memberName, "this", compiler), + getRemoveMemberListenerCode(handler, objectCode, memberName, "this", compiler)); } } catch (UnsupportedAttributeException e) { // ignore -- this is thrown for methods like toString(), for which there is no tracking and @@ -620,8 +651,8 @@ ClassDescriptor eventClass = DefaultObjectHandler.getEventClass(eventInfo.getListenerClass()); if (!methodExists) { methods.add(new JavaMethod(Modifier.PUBLIC, "void", methodName, - new JavaArgument[]{new JavaArgument(JAXXCompiler.getCanonicalName(eventClass), "event")}, null, - "propertyChange(null);", false)); + new JavaArgument[]{new JavaArgument(JAXXCompiler.getCanonicalName(eventClass), "event")}, null, + "propertyChange(null);", false)); } String code = objectCode + (eventInfo.getModelName() != null ? ".get" + StringUtils.capitalize(eventInfo.getModelName()) + "()" : ""); result.append("$bindingSources.put(\"").append(code).append("\", ").append(code).append(");").append(JAXXCompiler.getLineSeparator()); @@ -630,16 +661,16 @@ result.append(JAXXCompiler.getLineSeparator()); if (eventInfo.getModelName() != null) { result.append(getAddMemberListenerCode(handler, objectCode, "get" + StringUtils.capitalize(eventInfo.getModelName()), - JAXXUtil.class.getSimpleName() + ".getDataBindingUpdateListener(" + (compiler.getOutputClassName() + ".this") + ", " + constantId + ")", - compiler)); + JAXXUtil.class.getSimpleName() + ".getDataBindingUpdateListener(" + compiler.getOutputClassName() + ".this" + ", " + constantId + ")", + compiler)); } return result.toString(); } String propertyName = null; if (memberName.startsWith("get")) { - propertyName = Introspector.decapitalize(memberName.substring("get".length())); + propertyName = Introspector.decapitalize(memberName.substring(3)); } else if (memberName.startsWith("is")) { - propertyName = Introspector.decapitalize(memberName.substring("is".length())); + propertyName = Introspector.decapitalize(memberName.substring(2)); } else { try { handler.getBeanClass().getFieldDescriptor(memberName); @@ -690,8 +721,8 @@ if (!methodExists) { ClassDescriptor eventClass = DefaultObjectHandler.getEventClass(eventInfo.getListenerClass()); methods.add(new JavaMethod(Modifier.PUBLIC, "void", methodName, - new JavaArgument[]{new JavaArgument(JAXXCompiler.getCanonicalName(eventClass), "event")}, null, - "propertyChange(null);", false)); + new JavaArgument[]{new JavaArgument(JAXXCompiler.getCanonicalName(eventClass), "event")}, null, + "propertyChange(null);", false)); } try { String modelMemberName = eventInfo.getModelName() != null ? "get" + StringUtils.capitalize(eventInfo.getModelName()) : null; @@ -706,8 +737,8 @@ result.append("}").append(eol); if (eventInfo.getModelName() != null) { result.append(getRemoveMemberListenerCode(handler, objectCode, "get" + StringUtils.capitalize(eventInfo.getModelName()), - JAXXUtil.class.getSimpleName() + ".getDataBindingUpdateListener(" + compiler.getOutputClassName() + ".this, " + constantId + ")", - compiler)); + JAXXUtil.class.getSimpleName() + ".getDataBindingUpdateListener(" + compiler.getOutputClassName() + ".this, " + constantId + ")", + compiler)); } return result.toString(); } catch (NoSuchMethodException e) { Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/decorators/DefaultCompiledObjectDecorator.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/decorators/DefaultCompiledObjectDecorator.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/decorators/DefaultCompiledObjectDecorator.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -29,44 +29,115 @@ import jaxx.compiler.CompiledObjectDecorator; import jaxx.compiler.CompilerException; import jaxx.compiler.JAXXCompiler; +import jaxx.compiler.java.JavaField; import jaxx.compiler.java.JavaFile; import jaxx.compiler.java.JavaFileGenerator; +import jaxx.compiler.java.JavaMethod; import jaxx.compiler.script.ScriptInitializer; import jaxx.compiler.types.TypeManager; +import org.apache.commons.lang.StringUtils; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import java.lang.reflect.Modifier; import java.util.Map.Entry; /** * The default decorator to use on all compiled objects. - * + * * @author tchemit <chemit(a)codelutin.com> + * @plexus.component role-hint="default" role="jaxx.compiler.CompiledObjectDecorator" * @since 1.2 - * @plexus.component role-hint="default" role="jaxx.compiler.CompiledObjectDecorator" */ public class DefaultCompiledObjectDecorator implements CompiledObjectDecorator { + + /** Logger */ + protected static final Log log = + LogFactory.getLog(DefaultCompiledObjectDecorator.class); + @Override public String getName() { return "default"; } @Override - public void finalizeCompiler(JAXXCompiler compiler, CompiledObject root, CompiledObject object, JavaFile javaFile, String packageName, String className, String fullClassName) { + public void finalizeCompiler(JAXXCompiler compiler, + CompiledObject root, + CompiledObject object, + JavaFile javaFile, + String packageName, + String className, + String fullClassName) { - if (!object.isOverride() && !(object instanceof ScriptInitializer)) { - String id = object.getId(); + if (object instanceof ScriptInitializer) { + + // nothing to finalize + return; + } + + String fqn = JAXXCompiler.getCanonicalName(object); + + String id = object.getId(); + + if (log.isDebugEnabled()) { + log.debug("finalize " + id); + } + + if (object.isOverride()) { + + // add a specialized getter, only if type has changed + + if (object.getObjectClass().equals(object.getOverrideType())) { + + // type is the same do nothing + + return; + } + + // add a specialized getter + + String methodName = "get" + StringUtils.capitalize(id); + + String body = "return (" + fqn + ") " + id + ";"; + + if (log.isDebugEnabled()) { + log.debug("Add specialized getter " + methodName + " : " + body); + } + JavaMethod getter = JavaFileGenerator.newMethod(Modifier.PUBLIC, fqn, methodName, body, true); + javaFile.addMethod(getter); + } else { + int access = id.startsWith("$") ? Modifier.PRIVATE : Modifier.PROTECTED; - if (object == root) { - javaFile.addSimpleField(JavaFileGenerator.newField(access, className, id, false, "this")); + if (root.equals(object)) { + JavaField field = JavaFileGenerator.newField(access, className, id, false, "this"); + javaFile.addSimpleField(field); } else { - javaFile.addField(JavaFileGenerator.newField(access, JAXXCompiler.getCanonicalName(object), id, object.isOverride()), object.isJavaBean()); + + JavaField field = JavaFileGenerator.newField(access, fqn, id, object.isOverride()); + javaFile.addField(field, object.isJavaBean()); } } - if (!compiler.inlineCreation(object) && object != root) { + if (!compiler.inlineCreation(object) && !root.equals(object)) { javaFile.addMethod(JavaFileGenerator.newMethod(Modifier.PROTECTED, "void", object.getCreationMethodName(), getCreationCode(compiler, object), object.isOverride())); } + +// if (!object.isOverride() && !(object instanceof ScriptInitializer)) { +// String id = object.getId(); +// int access = id.startsWith("$") ? Modifier.PRIVATE : Modifier.PROTECTED; +// if (object == root) { +// JavaField field = JavaFileGenerator.newField(access, className, id, false, "this"); +// javaFile.addSimpleField(field); +// } else { +// JavaField field = JavaFileGenerator.newField(access, JAXXCompiler.getCanonicalName(object), id, object.isOverride()); +// javaFile.addField(field, object.isJavaBean()); +// } +// } +// +// if (!compiler.inlineCreation(object) && object != root) { +// javaFile.addMethod(JavaFileGenerator.newMethod(Modifier.PROTECTED, "void", object.getCreationMethodName(), getCreationCode(compiler, object), object.isOverride())); +// } } @Override @@ -145,7 +216,18 @@ @Override public boolean createInitializer(JAXXCompiler compiler, CompiledObject root, CompiledObject object, StringBuffer code, boolean lastWasMethodCall) { String eol = JAXXCompiler.getLineSeparator(); - if (object == root) { + + if (object instanceof ScriptInitializer) { + + // initializer has special direct treatment : can not be in a method + // just push code to compiler + code.append(getCreationCode(compiler, object)); + + // nothing to initialize of a script + return lastWasMethodCall; + } + + if (root.equals(object)) { String rootCode = root.getInitializationCode(compiler); if (rootCode != null && rootCode.length() > 0) { code.append("// inline creation of ").append(object.getId()).append(eol); Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/CompileFirstPassTask.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/CompileFirstPassTask.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/CompileFirstPassTask.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -44,30 +44,25 @@ /** Logger */ private static final Log log = LogFactory.getLog(CompileFirstPassTask.class); - private static final long serialVersionUID = -1L; - + /** Task name */ public static final String TASK_NAME = "CompileFirstPass"; public CompileFirstPassTask() { super(TASK_NAME); } - protected boolean compiled; - @Override public boolean perform(JAXXEngine engine) throws Exception { boolean success = true; - compiled = true; - while (compiled) { - // at each round, says nothing has to be compiled - // the method treateFile will if required change the state - compiled = false; + JAXXCompilerFile[] undone = engine.getFilesToCompile(); - JAXXCompilerFile[] files = engine.getCompilingFiles(); + while (undone.length > 0) { - for (JAXXCompilerFile jaxxFile : files) { + // start a new round of compilation + for (JAXXCompilerFile jaxxFile : undone) { + boolean successForFile = treateFile(engine, jaxxFile); if (!successForFile) { @@ -76,6 +71,9 @@ success = false; } } + + // prepare next round + undone = engine.getFilesToCompile(); } return success; } @@ -86,32 +84,21 @@ boolean isVerbose = engine.isVerbose(); if (isVerbose) { - log.info(getName() + " for " + jaxxFile.getClassName()); + log.info("start " + jaxxFile.getClassName()); } - JAXXCompiler compiler = jaxxFile.getCompiler(); - - if (compiler != null) { - - // file already registred - return true; - } - boolean success = true; - // mark to have another round to treate new files - compiled = true; - CompilerConfiguration configuration = engine.getConfiguration(); - compiler = engine.newCompiler(jaxxFile); + JAXXCompiler compiler = engine.newCompiler(jaxxFile); addStartProfileTime(engine, compiler); compiler.compileFirstPass(); if (configuration.isAutoImportCss() && !compiler.isIdentCssFound()) { - // check if can add ident css file + // check if we can add ident css file File cssFile = jaxxFile.getCssFile(); Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/CompileSecondPassTask.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/CompileSecondPassTask.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/CompileSecondPassTask.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -24,15 +24,12 @@ */ package jaxx.compiler.tasks; -import jaxx.compiler.CompilerException; import jaxx.compiler.JAXXCompiler; import jaxx.compiler.JAXXCompilerFile; import jaxx.compiler.JAXXEngine; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; -import java.util.Arrays; - /** * Task to execute the Second round of compile. * @@ -45,8 +42,7 @@ private static final Log log = LogFactory.getLog(CompileSecondPassTask.class); - private static final long serialVersionUID = -1L; - + /** Task name */ public static final String TASK_NAME = "CompileSecondPass"; public CompileSecondPassTask() { @@ -58,16 +54,20 @@ boolean success = true; boolean isVerbose = engine.isVerbose(); - JAXXCompilerFile[] files = engine.getCompilingFiles(); + // check all files are attached to a compiler + checkAllFilesCompiled(engine); + JAXXCompilerFile[] files = engine.getCompiledFiles(); + for (JAXXCompilerFile jaxxFile : files) { String className = jaxxFile.getClassName(); if (isVerbose) { - log.info(getName() + " for " + className); + log.info("start " + className); } - JAXXCompiler compiler = getSafeCompiler(jaxxFile); + JAXXCompiler compiler = jaxxFile.getCompiler(); + addStartProfileTime(engine, compiler); if (log.isDebugEnabled()) { log.debug("runInitializers for " + className); @@ -89,15 +89,7 @@ } } - // check that compilation set was not altered during pass 2 - JAXXCompilerFile[] jaxxFilesClone = engine.getCompilingFiles(); - if (jaxxFilesClone.length > files.length) { - throw new CompilerException( - "Internal error: compilation set altered during pass" + - " 2 (was " + Arrays.toString(jaxxFilesClone) + - ", modified to " + Arrays.toString(files) + ")" - ); - } return success; } + } \ No newline at end of file Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/FinalizeTask.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/FinalizeTask.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/FinalizeTask.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -43,8 +43,7 @@ /** Logger */ private static final Log log = LogFactory.getLog(FinalizeTask.class); - private static final long serialVersionUID = -1L; - + /** Task name */ public static final String TASK_NAME = "Finalize"; public FinalizeTask() { @@ -56,12 +55,17 @@ boolean success = true; boolean isVerbose = engine.getConfiguration().isVerbose(); - for (JAXXCompilerFile jaxxFile : engine.getCompilingFiles()) { + // check all files are attached to a compiler + checkAllFilesCompiled(engine); + JAXXCompilerFile[] files = engine.getCompiledFiles(); + + for (JAXXCompilerFile jaxxFile : files) { + if (isVerbose) { - log.info(getName() + " for " + jaxxFile.getClassName()); + log.info("start " + jaxxFile.getClassName()); } - JAXXCompiler compiler = getSafeCompiler(jaxxFile); + JAXXCompiler compiler = jaxxFile.getCompiler(); addStartProfileTime(engine, compiler); compiler.finalizeCompiler(); addEndProfileTime(engine, compiler); Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/GenerateTask.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/GenerateTask.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/GenerateTask.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -42,8 +42,7 @@ /** Logger */ private static final Log log = LogFactory.getLog(GenerateTask.class); - private static final long serialVersionUID = -1L; - + /** Task name */ public static final String TASK_NAME = "Generate"; public GenerateTask() { @@ -52,6 +51,10 @@ @Override public boolean perform(JAXXEngine engine) throws Exception { + + // check all files are attached to a compiler + checkAllFilesCompiled(engine); + boolean success = true; boolean verbose = engine.isVerbose(); @@ -59,13 +62,15 @@ JavaFileGenerator generator = new JavaFileGenerator(JAXXCompiler.getLineSeparator(), verbose); - for (JAXXCompilerFile jaxxFile : engine.getCompilingFiles()) { + JAXXCompilerFile[] files = engine.getCompiledFiles(); + + for (JAXXCompilerFile jaxxFile : files) { if (verbose) { - log.info(getName() + " for " + jaxxFile.getClassName()); + log.info("start " + jaxxFile.getClassName()); } - JAXXCompiler compiler = getSafeCompiler(jaxxFile); + JAXXCompiler compiler = jaxxFile.getCompiler(); addStartProfileTime(engine, compiler); compiler.generate(generator); addEndProfileTime(engine, compiler); Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/InitTask.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/InitTask.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/InitTask.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -38,13 +38,12 @@ */ public class InitTask extends JAXXEngineTask { - private static final long serialVersionUID = -1L; - - public static final String TASK_NAME = "Init"; - /** Logger */ private static final Log log = LogFactory.getLog(InitTask.class); + /** Task name */ + public static final String TASK_NAME = "Init"; + public InitTask() { super(TASK_NAME); } @@ -76,7 +75,7 @@ } if (configuration.isVerbose()) { - log.info("will init " + JAXXFactory.class.getName()); + log.info("Will init " + JAXXFactory.class.getName()); } JAXXFactory.initFactory(); Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/JAXXEngineTask.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/JAXXEngineTask.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/JAXXEngineTask.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -24,13 +24,10 @@ */ package jaxx.compiler.tasks; -import jaxx.compiler.CompilerException; import jaxx.compiler.JAXXCompiler; import jaxx.compiler.JAXXCompilerFile; import jaxx.compiler.JAXXEngine; -import java.io.Serializable; - /** * Base class to implement a task to be launched by a {@link JAXXEngine}. * <p/> @@ -39,49 +36,51 @@ * @author tchemit <chemit(a)codelutin.com> * @since 2.0.2 */ -public abstract class JAXXEngineTask implements Serializable { +public abstract class JAXXEngineTask { + /** Task name */ private final String name; public JAXXEngineTask(String name) { this.name = name; } + /** + * Performs the task on the given {@code engine}. + * + * @param engine the engine to use + * @return {@code false} if task failed (with no exception), {@code true} otherwise. + * @throws Exception if any error + */ public abstract boolean perform(JAXXEngine engine) throws Exception; - public String getName() { return name; } + /** + * Checks the engine does not have any more files to discover. + * + * @param engine the engine to test + * @throws IllegalStateException if there is still some files to discover. + */ + protected void checkAllFilesCompiled(JAXXEngine engine) throws IllegalStateException { + JAXXCompilerFile[] undone = engine.getFilesToCompile(); + + if (undone.length > 0) { + + throw new IllegalStateException("Can not start '" + getName() + "', there is still files to process in '" + CompileFirstPassTask.TASK_NAME); + } + } + protected void addStartProfileTime(JAXXEngine engine, JAXXCompiler compiler) { engine.addProfileTime(compiler, name + "_start"); } + protected void addEndProfileTime(JAXXEngine engine, JAXXCompiler compiler) { engine.addProfileTime(compiler, name + "_end"); } - - protected JAXXCompiler getSafeCompiler(JAXXCompilerFile engine) { - JAXXCompiler compiler = engine.getCompiler(); - if (compiler == null) { - throw new CompilerException( - "Internal error: could not find compiler for " + - engine.getClassName() + " during task [" + getName() + "]" - ); - } - - return compiler; - } -// -// protected JAXXCompiler getSafeCompiler(JAXXEngine engine, -// String className) { -// JAXXCompiler compiler = engine.getCompiler( -// className, -// "Internal error: could not find compiler for " + className + -// " during task [" + getName() + "]"); -// return compiler; -// } } Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/ProfileTask.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/ProfileTask.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/ProfileTask.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -39,8 +39,7 @@ /** Logger */ private static final Log log = LogFactory.getLog(ProfileTask.class); - private static final long serialVersionUID = -1L; - + /** Task name */ public static final String TASK_NAME = "Profile"; public ProfileTask() { Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/StyleSheetTask.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/StyleSheetTask.java 2010-05-18 18:04:03 UTC (rev 1921) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tasks/StyleSheetTask.java 2010-05-18 18:08:14 UTC (rev 1922) @@ -41,8 +41,7 @@ /** Logger */ private static final Log log = LogFactory.getLog(StyleSheetTask.class); - private static final long serialVersionUID = -1L; - + /** Task name */ public static final String TASK_NAME = "StyleSheet"; public StyleSheetTask() { @@ -51,18 +50,26 @@ @Override public boolean perform(JAXXEngine engine) throws Exception { + + // check all files are attached to a compiler + checkAllFilesCompiled(engine); + boolean success = true; boolean isVerbose = engine.getConfiguration().isVerbose(); - JAXXCompilerFile[] files = engine.getCompilingFiles(); + + JAXXCompilerFile[] files = engine.getCompiledFiles(); + for (JAXXCompilerFile jaxxFile : files) { String className = jaxxFile.getClassName(); if (isVerbose) { - log.info(getName() + " for " + className); + log.info("start " + className); } - JAXXCompiler compiler = getSafeCompiler(jaxxFile); + + JAXXCompiler compiler = jaxxFile.getCompiler(); addStartProfileTime(engine, compiler); compiler.applyStylesheets(); addEndProfileTime(engine, compiler); + if (compiler.isFailed()) { success = false; }

1 0

r1921 - trunk/jaxx-compiler/src/main/java/jaxx/compiler
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 20:04:03 +0200 (Tue, 18 May 2010) New Revision: 1921 Url: http://nuiton.org/repositories/revision/jaxx/1921 Log: - clean code - Evolution #626: generate generics on overriden object Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/CompiledObject.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/CompiledObject.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/CompiledObject.java 2010-05-18 17:59:37 UTC (rev 1920) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/CompiledObject.java 2010-05-18 18:04:03 UTC (rev 1921) @@ -147,18 +147,6 @@ return child; } -// public void setChild(CompiledObject child) { -// this.child = child; -// } -// -// public String getConstraints() { -// return constraints; -// } -// -// public void setConstraints(String constraints) { -// this.constraints = constraints; -// } - public String getChildJavaCode() { return childJavaCode; } @@ -167,14 +155,6 @@ this.childJavaCode = childJavaCode; } -// public String getDelegateCode() { -// return delegateCode; -// } -// -// public void setDelegateCode(String delegateCode) { -// this.delegateCode = delegateCode; -// } - public void addToAdditionCode(StringBuffer buffer, boolean isRootObject) { //TC-20091026 do not prefix if on root object String prefix; @@ -255,7 +235,7 @@ throw new NullPointerException(); } this.objectClass = objectClass; - this.childs = new ArrayList<ChildRef>(); + childs = new ArrayList<ChildRef>(); } public static boolean isValidID(String id) { @@ -397,16 +377,17 @@ String result = javaCode; if (isOverride()) { // handle cases where object is overridden to be a different class - result = "((" + JAXXCompiler.getCanonicalName(getObjectClass()) + ") " + javaCode + ")"; + result = "((" + JAXXCompiler.getCanonicalName(this) + ") " + javaCode + ")"; } return result; } public String getJavaCodeForProperty(String property) { - if (!override) { + if (!isOverride()) { return javaCode; } - String result = "((" + JAXXCompiler.getCanonicalName(getObjectClass()) + ") " + javaCode + ")"; + String result = "((" + JAXXCompiler.getCanonicalName(this) + ") " + javaCode + ")"; +// String result = "((" + JAXXCompiler.getCanonicalName(getObjectClass()) + ") " + javaCode + ")"; String methodName = StringUtils.capitalize(property); try { @@ -501,8 +482,8 @@ */ public void appendInitializationCode(String code) { if (!code.isEmpty()) { - this.initializationCode.append(code); - this.initializationCode.append(JAXXCompiler.getLineSeparator()); + initializationCode.append(code); + initializationCode.append(JAXXCompiler.getLineSeparator()); } } @@ -515,8 +496,8 @@ */ public void appendAdditionCode(String code) { if (!code.isEmpty()) { - this.additionCode.append(code); - this.additionCode.append(JAXXCompiler.getLineSeparator()); + additionCode.append(code); + additionCode.append(JAXXCompiler.getLineSeparator()); } } @@ -594,7 +575,7 @@ compiler.registerEventHandler(handler); eventHandlers.add(handler); - if (getJavaCode().indexOf(".") != -1) { + if (getJavaCode().contains(".")) { // object lives in another JAXX file and consequently its initialization code won't be output compiler.appendInitializerCode(getInitializationCode(handler, compiler)); } @@ -714,12 +695,12 @@ public void finalizeCompiler(JAXXCompiler compiler) { - List<CompiledObject.ChildRef> refList = getChilds(); + List<ChildRef> refList = getChilds(); if (refList != null && !refList.isEmpty()) { // compute additionCode for all childs StringBuffer buffer = new StringBuffer(); for (ChildRef childRef : refList) { - childRef.addToAdditionCode(buffer, compiler.getRootObject() == this); + childRef.addToAdditionCode(buffer, equals(compiler.getRootObject())); } additionCode = buffer.append(additionCode); }

1 0

r1920 - trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/resolvers
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 19:59:37 +0200 (Tue, 18 May 2010) New Revision: 1920 Url: http://nuiton.org/repositories/revision/jaxx/1920 Log: Anomalie #624: Interface methods are not loaded in ClassDescriptor Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/resolvers/ClassDescriptorResolverFromJavaFile.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/resolvers/ClassDescriptorResolverFromJavaFile.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/resolvers/ClassDescriptorResolverFromJavaFile.java 2010-05-18 17:59:04 UTC (rev 1919) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/resolvers/ClassDescriptorResolverFromJavaFile.java 2010-05-18 17:59:37 UTC (rev 1920) @@ -487,6 +487,10 @@ int modifiers = getModifiers(node); + if (isInterface) { + + modifiers = Modifier.PUBLIC; + } if (log.isDebugEnabled()) { log.debug("method [" + name + "] modifiers == " + Modifier.toString(modifiers)); }

1 0

r1919 - trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 19:59:04 +0200 (Tue, 18 May 2010) New Revision: 1919 Url: http://nuiton.org/repositories/revision/jaxx/1919 Log: remove very old comment code Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java 2010-05-18 17:58:27 UTC (rev 1918) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java 2010-05-18 17:59:04 UTC (rev 1919) @@ -448,9 +448,6 @@ javaFile.addBodyCode(compiler.getBodyCode().toString()); -// javaFile.addMethod(REGISTER_DATA_BINDING_METHOD); -// javaFile.addMethod(GET_DATA_BINDING_METHOD); - if (hasDataBindings) { // create the $registerDefaultBindings method @@ -480,9 +477,6 @@ } javaFile.addMethod(createCompleteSetupMethod(compiler, javaFile)); -// javaFile.addMethod(createApplyDataBindingMethod(superclassIsJAXXObject, hasDataBindings)); -// javaFile.addMethod(createProcessDataBindingMethod(superclassIsJAXXObject, hasDataBindings)); -// javaFile.addMethod(createRemoveDataBindingMethod(superclassIsJAXXObject, hasDataBindings)); addEventHandlers(compiler, javaFile); } @@ -495,7 +489,6 @@ protected JavaMethod createRegisterDefaultBindingsMethod(JAXXCompiler compiler) { DataBinding[] bindings = compiler.getBindingHelper().getDataBindings(); StringBuilder initCode = new StringBuilder(); -// if (bindings.legth > 0) { String eol = JAXXCompiler.getLineSeparator(); JavaFileGenerator generator = new JavaFileGenerator(eol, true); @@ -597,10 +590,13 @@ protected void addEventHandlers(JAXXCompiler compiler, JavaFile javaFile) { for (Map.Entry<String, Map<ClassDescriptor, List<EventHandler>>> e1 : compiler.getEventHandlers().entrySet()) { + // outer loop is iterating over different objects (well, technically, different Java expressions) for (Map.Entry<ClassDescriptor, List<EventHandler>> e2 : e1.getValue().entrySet()) { + // iterate over different types of listeners for this particular object (MouseListener, ComponentListener, etc.) for (EventHandler handler : e2.getValue()) { + // iterate over individual event handlers of a single type String methodName = compiler.getEventHandlerMethodName(handler); MethodDescriptor listenerMethod = handler.getListenerMethod(); @@ -763,102 +759,33 @@ return result.toString(); } - // protected JavaMethod createApplyDataBindingMethod(boolean superclassIsJAXXObject, boolean haveBinding) { protected JavaMethod createApplyDataBindingMethod() { StringBuilder buffer = new StringBuilder(); String eol = JAXXCompiler.getLineSeparator(); -// if (haveBinding) { -// if (superclassIsJAXXObject) buffer.append("if (" + FIELD_NAME_ALL_COMPONENTS_CREATED + " && " + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); -// else { -// buffer.append("if (" + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); -// } buffer.append(" " + FIELD_NAME_$BINDINGS + ".get(" + PARAMETER_NAME_$BINDING + ")." + METHOD_NAME_APPLY_DATA_BINDING + "();").append(eol); buffer.append("}"); - -// } -// if (superclassIsJAXXObject) { -// -// if (haveBinding) { -// buffer.append(" else {").append(eol).append(" "); -// } -// buffer.append("super." + METHOD_NAME_APPLY_DATA_BINDING + "(" + PARAMETER_NAME_$BINDING + ");").append(eol); -// -// if (haveBinding) { -// buffer.append(" return;").append(eol).append("}"); -// } -// -// } - -// if (haveBinding) { -// if (superclassIsJAXXObject) -// buffer.append("if (" + FIELD_NAME_ALL_COMPONENTS_CREATED + " && " + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); -// else { -// buffer.append("if (" + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); -// } -// buffer.append(" " + FIELD_NAME_$BINDINGS + ".get(" + PARAMETER_NAME_$BINDING + ")." + METHOD_NAME_APPLY_DATA_BINDING + "();").append(eol); -// buffer.append("}"); -// -// } -// if (superclassIsJAXXObject) { -// -// if (haveBinding) { -// buffer.append(" else {").append(eol).append(" "); -// } -// buffer.append("super." + METHOD_NAME_APPLY_DATA_BINDING + "(" + PARAMETER_NAME_$BINDING + ");").append(eol); -// -// if (haveBinding) { -// buffer.append(" return;").append(eol).append("}"); -// } -// -// } buffer.append(eol).append(METHOD_NAME_PROCESS_DATA_BINDING + "(" + PARAMETER_NAME_$BINDING + ");"); return newMethod(PUBLIC, TYPE_VOID, METHOD_NAME_APPLY_DATA_BINDING, buffer.toString(), true, new JavaArgument(TYPE_STRING, PARAMETER_NAME_$BINDING)); } protected JavaMethod createRemoveDataBindingMethod() { -// protected JavaMethod createRemoveDataBindingMethod(boolean superclassIsJAXXObject, boolean haveBinding) { StringBuilder buffer = new StringBuilder(); String eol = JAXXCompiler.getLineSeparator(); buffer.append("if (" + FIELD_NAME_ALL_COMPONENTS_CREATED + " && " + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); buffer.append(" " + FIELD_NAME_$BINDINGS + ".get(" + PARAMETER_NAME_$BINDING + ")." + METHOD_NAME_REMOVE_DATA_BINDING + "();").append(eol); buffer.append("}"); - -// if (haveBinding) { -// -// if (superclassIsJAXXObject) { -// buffer.append("if (" + FIELD_NAME_ALL_COMPONENTS_CREATED + " && " + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); -// } else { -// buffer.append("if (" + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); -// } -// buffer.append(" " + FIELD_NAME_$BINDINGS + ".get(" + PARAMETER_NAME_$BINDING + ")." + METHOD_NAME_REMOVE_DATA_BINDING + "();").append(eol); -// buffer.append("}"); -// -// -// } -// if (superclassIsJAXXObject) { -// -// if (haveBinding) { -// buffer.append(" else {").append(eol).append(" "); -// } -// buffer.append("super." + METHOD_NAME_REMOVE_DATA_BINDING + "(" + PARAMETER_NAME_$BINDING + ");").append(eol); -// -// if (haveBinding) { -// buffer.append("}"); -// } -// } return newMethod(PUBLIC, TYPE_VOID, METHOD_NAME_REMOVE_DATA_BINDING, buffer.toString(), true, new JavaArgument(TYPE_STRING, PARAMETER_NAME_$BINDING)); } protected JavaMethod createProcessDataBindingMethod() { -// protected JavaMethod createProcessDataBindingMethod(boolean superclassIsJAXXObject, boolean haveBinding) { StringBuffer code = new StringBuffer(); String eol = JAXXCompiler.getLineSeparator(); - //boolean superclassIsJAXXObject = ClassDescriptorHelper.getClassDescriptor(JAXXObject.class).isAssignableFrom(compiler.getRootObject().getObjectClass()); + // the force parameter forces the update to happen even if it is already in activeBindings. This // is used on superclass invocations b/c by the time the call gets to the superclass, it is already // marked active and would otherwise be skipped @@ -874,33 +801,6 @@ code.append("} finally {").append(eol); code.append(" " + FIELD_NAME_$ACTIVE_BINDINGS + ".remove(" + PARAMETER_NAME_$BINDING + ");").append(eol); code.append("}").append(eol); - -// if (haveBinding) { -// code.append(" if (!$force && " + FIELD_NAME_$ACTIVE_BINDINGS + ".contains(" + PARAMETER_NAME_$BINDING + ")) { "); -// code.append(eol); -// code.append(" return;").append(eol); -// code.append("}").append(eol); -// code.append(FIELD_NAME_$ACTIVE_BINDINGS + ".add(" + PARAMETER_NAME_$BINDING + ");").append(eol); -// code.append("try {").append(eol); -// if (superclassIsJAXXObject) { -// code.append(" if (" + FIELD_NAME_ALL_COMPONENTS_CREATED + " && " + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); -// } else { -// code.append(" if (" + FIELD_NAME_$BINDINGS + ".containsKey(" + PARAMETER_NAME_$BINDING + ")) {").append(eol); -// } -// code.append(" " + FIELD_NAME_$BINDINGS + ".get(" + PARAMETER_NAME_$BINDING + ")." + METHOD_NAME_PROCESS_DATA_BINDING + "();").append(eol); -// code.append(" }"); -// if (superclassIsJAXXObject) { -// code.append(" else {").append(eol); -// code.append(" super." + METHOD_NAME_PROCESS_DATA_BINDING + "(" + PARAMETER_NAME_$BINDING + ", true);").append(eol); -// code.append(" }"); -// } -// code.append(eol); -// code.append("} finally {").append(eol); -// code.append(" " + FIELD_NAME_$ACTIVE_BINDINGS + ".remove(" + PARAMETER_NAME_$BINDING + ");").append(eol); -// code.append("}").append(eol); -// } else if (superclassIsJAXXObject) { -// code.append("super." + METHOD_NAME_PROCESS_DATA_BINDING + "(" + PARAMETER_NAME_$BINDING + ", true);").append(eol); -// } return newMethod(PUBLIC, TYPE_VOID, METHOD_NAME_PROCESS_DATA_BINDING, code.toString(), true, new JavaArgument(TYPE_STRING, PARAMETER_NAME_$BINDING), new JavaArgument(TYPE_BOOLEAN, "$force")); }

1 0

r1918 - trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 19:58:27 +0200 (Tue, 18 May 2010) New Revision: 1918 Url: http://nuiton.org/repositories/revision/jaxx/1918 Log: improve logs (and prepare form mojo showClassLoading flga) Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/ClassDescriptorHelper.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/ClassDescriptorHelper.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/ClassDescriptorHelper.java 2010-05-18 12:41:06 UTC (rev 1917) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/ClassDescriptorHelper.java 2010-05-18 17:58:27 UTC (rev 1918) @@ -62,6 +62,8 @@ /** Logger */ private static final Log log = LogFactory.getLog(ClassDescriptorHelper.class); + private static boolean SHOW_LOADING = log.isDebugEnabled(); + private static Map<String, ClassDescriptor> descriptors = new HashMap<String, ClassDescriptor>(); @@ -97,6 +99,16 @@ } } + public static URL getURL(ClassLoader classLoader, String className, String clasifier) { + String relativePath = className.replaceAll("\\.", "/"); + String path = relativePath + "." + clasifier; + if (log.isDebugEnabled()) { + log.debug("Path to search : " + path); + } + URL url = classLoader.getResource(path); + return url; + } + public static ClassDescriptor getClassDescriptor( String className, ClassLoader classLoader) throws ClassNotFoundException { @@ -107,12 +119,19 @@ ClassDescriptor result = descriptors.get(className); if (result != null) { + + // found in cache + if (log.isTraceEnabled()) { + log.trace("resolved " + result + " from cache."); + } return result; } JAXXEngine engine = JAXXFactory.isEngineRegistred() ? JAXXFactory.getEngine() : null; + String relativePathPattern = ".*";// + className + ".*"; // used to ensure that the located resource has the right character cases + if (engine != null) { JAXXCompilerFile file = engine.getJAXXCompilerFile(className); @@ -121,6 +140,9 @@ // use the symbole table of this jaxx file on computation + if (SHOW_LOADING) { + log.info("from JAXXFile " + file.getJaxxFile()); + } result = getClassDescriptor0( ClassDescriptorResolverFromJaxxFile.class, className, @@ -128,6 +150,10 @@ classLoader ); + if (log.isDebugEnabled()) { + log.debug("[" + className + "] loaded"); + } + return result; } } @@ -135,12 +161,9 @@ // the class is not in this compile set, try to have it from java // sources or classes. - String relativePath = className.replaceAll("\\.", "/"); - String relativePathPattern = ".*";// + className + ".*"; // used to ensure that the located resource has the right character cases - // find the most recently updated source for the class -- Java source, JAXX source, or compiled class file long javaLastModified = -1; - URL javaFile = classLoader.getResource(relativePath + ".java"); + URL javaFile = getURL(classLoader, className, "java"); if (javaFile != null && javaFile.toString().startsWith("file:") && javaFile.toString().matches(relativePathPattern)) { @@ -151,7 +174,7 @@ } long classLastModified = -1; - URL classFile = classLoader.getResource(relativePath + ".class"); + URL classFile = getURL(classLoader, className, "class"); if (classFile != null && classFile.toString().startsWith("file:") && classFile.toString().matches(relativePathPattern)) { @@ -166,6 +189,10 @@ // java file exist and it is the last modified file, so use it + if (SHOW_LOADING) { + log.info("from JavaFile " + javaFile); + } + result = getClassDescriptor0( ClassDescriptorResolverFromJavaFile.class, className, @@ -173,11 +200,18 @@ classLoader ); + if (log.isDebugEnabled()) { + log.debug("[" + className + "] loaded"); + } return result; } // use the class ... + if (SHOW_LOADING) { + log.info("from class " + className); + } + result = getClassDescriptor0( ClassDescriptorResolverFromJavaClass.class, className, @@ -185,11 +219,14 @@ classLoader ); + if (log.isDebugEnabled()) { + log.debug("[" + className + "] loaded"); + } + if (result != null) { return result; } - // can NOT come here, means could not find result... throw new IllegalStateException(

1 0

r1917 - trunk/jaxx-demo/src/main/java/jaxx/demo
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 14:41:06 +0200 (Tue, 18 May 2010) New Revision: 1917 Url: http://nuiton.org/repositories/revision/jaxx/1917 Log: show log when error before going in ErrorDialogUI Modified: trunk/jaxx-demo/src/main/java/jaxx/demo/RunDemo.java Modified: trunk/jaxx-demo/src/main/java/jaxx/demo/RunDemo.java =================================================================== --- trunk/jaxx-demo/src/main/java/jaxx/demo/RunDemo.java 2010-05-18 05:49:08 UTC (rev 1916) +++ trunk/jaxx-demo/src/main/java/jaxx/demo/RunDemo.java 2010-05-18 12:41:06 UTC (rev 1917) @@ -143,8 +143,8 @@ handler.displayUI(ui, config.getDemoPath()); } catch (Exception e) { + log.error(e.getMessage(), e); ErrorDialogUI.showError(e); - log.error(e.getMessage(), e); System.exit(1); } }

1 0

r1916 - trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 07:49:08 +0200 (Tue, 18 May 2010) New Revision: 1916 Url: http://nuiton.org/repositories/revision/jaxx/1916 Log: Anomalie #622: Register bindings when all components are initialized Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java 2010-05-18 05:45:39 UTC (rev 1915) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java 2010-05-18 05:49:08 UTC (rev 1916) @@ -662,12 +662,12 @@ code.append("if (" + FIELD_NAME_ALL_COMPONENTS_CREATED + " || !" + FIELD_NAME_CONTEXT_INITIALIZED + ") {").append(eol); code.append(" return;").append(eol); code.append("}").append(eol); - // register bindings before anything else - if (nbBindings > 0) { - // ajout invocation a la methode d'enregistrement des bindings - code.append("// registers ").append(nbBindings).append(" data bindings").append(eol); - code.append(METHOD_NAME_$REGISTER_DEFAULT_BINDINGS + "();").append(eol); - } +// // register bindings before anything else +// if (nbBindings > 0) { +// // ajout invocation a la methode d'enregistrement des bindings +// code.append("// registers ").append(nbBindings).append(" data bindings").append(eol); +// code.append(METHOD_NAME_$REGISTER_DEFAULT_BINDINGS + "();").append(eol); +// } // String dataBindingsCode = createRegisterDefaultBindingsMethod(compiler); // if (dataBindingsCode != null && !dataBindingsCode.isEmpty()) { // code.append(dataBindingsCode).append(eol); @@ -691,6 +691,12 @@ if (compiler.getInitializer().length() > 0) { code.append(compiler.getInitializer()); } + // register bindings before anything else + if (nbBindings > 0) { + // ajout invocation a la methode d'enregistrement des bindings + code.append("// registers ").append(nbBindings).append(" data bindings").append(eol); + code.append(METHOD_NAME_$REGISTER_DEFAULT_BINDINGS + "();").append(eol); + } code.append(METHOD_NAME_$COMPLETE_SETUP + "();").append(eol); return newMethod(PRIVATE, TYPE_VOID, METHOD_NAME_$INITIALIZE, code.toString(), false); }

1 0

r1915 - trunk/jaxx-compiler/src/main/java/jaxx/compiler/script
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 07:45:39 +0200 (Tue, 18 May 2010) New Revision: 1915 Url: http://nuiton.org/repositories/revision/jaxx/1915 Log: change log level Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/script/ScriptManager.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/script/ScriptManager.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/script/ScriptManager.java 2010-05-18 05:45:17 UTC (rev 1914) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/script/ScriptManager.java 2010-05-18 05:45:39 UTC (rev 1915) @@ -326,8 +326,8 @@ name = name.substring(0, name.length() - 1).trim(); } String className = declarationTokens[declarationTokens.length - 2]; - if (log.isInfoEnabled()) { - log.info("Found type : "+className +": ()"+ Arrays.toString(declarationTokens)); + if (log.isDebugEnabled()) { + log.debug("Found type : " + className +" : "+ Arrays.toString(declarationTokens)); } String type = TagManager.resolveClassName(className, compiler); if (type == null) {

1 0

r1914 - trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect
by tchemit＠users.nuiton.org May 18, 2010

May 18, 2010

Author: tchemit Date: 2010-05-18 07:45:17 +0200 (Tue, 18 May 2010) New Revision: 1914 Url: http://nuiton.org/repositories/revision/jaxx/1914 Log: improve error when type is not found Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/FieldDescriptor.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/FieldDescriptor.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/FieldDescriptor.java 2010-05-18 03:48:55 UTC (rev 1913) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/FieldDescriptor.java 2010-05-18 05:45:17 UTC (rev 1914) @@ -44,7 +44,9 @@ try { return ClassDescriptorHelper.getClassDescriptor(type, getClassLoader()); } catch (ClassNotFoundException e) { - throw new RuntimeException(e); + throw new RuntimeException("Type not found for field " + this, e); + } catch (Exception e) { + throw new RuntimeException("Type not found for field " + this, e); } }

1 0