Bugs introduced that affected jde-import-all

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/[email protected]/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