I found two bugs when using `jde-import-all': 1. Classes not being imported. 2. Additional lines inserted when importing classes.
The first bug introduced in 181 when adding this to jde.el (autoload 'wisent-java-default-setup "wisent-java" "Hook run to setup Semantic in `java-mode'." nil nil) which comes from: http://www.mail-archive.com/j...@sunsite.dk/msg07334.html Looks like the 1.5 implementation is buggy as the URL states it might be. Perhaps it will work better in the newer cedet, but for now it doesn't make sense to break a major feature like `jde-import-all'. The second bug was introduced in 183 and inserts lines around inserted imports. Here's the comment for this: Also fixed a bug where the import insertion point did not go to the next line after the last import. so if you had a comment on the last import, jde would insert before that comment... This all highlights the need for some kind of automated testing. Also, these bugs took many hours to track down and fix so please do some basic testing after making big changes like adding 1.5 parsing cedet code and inner class importing. Thanks to all of our for your contributions and hard work. -- Paul Landes landes at mailc dot net ------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev _______________________________________________ jdee-devel mailing list jdee-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jdee-devel
