Hi Alessandro! The patch for Tree is applied to SVN, revision 4441.
The other patch is IMHO a bit problematic. We don't want to compress names in the global scope. However other parts are looking good. I've applied the complete patch but ignored the part which runs through the global namespace. I hope the table keeps working :) Cheers, Sebastian Alessandro Sala schrieb: > Hi Jim, Sebastian >> qx.ui.table.TablePane[127] >> 007249: Modification of property "visibleRowCount" failed with exception: >> TypeError - '$f' is null or not an object (#-2146823281 > It seems the error is due to some problem in the new variableoptimizer > module: when a property of > an object is used as an accessor (because it is an array-type or because > it follows a function or method call) > and at the same time there is an optimized variable with the same name, > the property gets substituted with > the optimized variable name, while it should not. In effect this problem > shows up in various tests, not only in Table_5. > > While analyzing the problem I also found two other bugs: > - variableoptimizer doesn't substitute function names with their > optimized value in the global scope > - in the tree module, the replaceChild() method doesn't assign the > "parent" property to newChild > > I attach two patches which correct the above problems, so that > Sebastian can > evaluate them, though I am not entirely satisfied with the > variableoptimizer patch since > it explicitly handles the above two special cases while I suspect there > can be other, > but for now it works ok. > > Cheers, > Alessandro > > > ------------------------------------------------------------------------ > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get the chance to share your > opinions on IT & business topics through brief surveys -- and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > > ------------------------------------------------------------------------ > > _______________________________________________ > qooxdoo-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ qooxdoo-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
