Fix regressions and further tweak previous java completion patch.

Fix inner class regression, adapt test suite, turn off nospace for
single class completions, protect -o nospace for bash < 4.

completions/java

@@ -72,11 +72,14 @@ _java_classes()
             fi
 
         elif [ -d $i ]; then
-            compopt -o nospace
+            compopt -o nospace &>/dev/null
             COMPREPLY=( "${COMPREPLY[@]}"
-                 $( compgen -d -- $i/$cur | sed -e "s|^$i/\(.*\)|\1.|" )
-                 $( compgen -f -X '!*.class' -- $i/$cur | sed -e "s|^$i/||" )
+                $( compgen -d -- "$i/$cur" | sed -e "s|^$i/\(.*\)|\1.|" )
+                $( compgen -f -X '!*.class' -- "$i/$cur" | \
+                    sed -e '/\$/d' -e "s|^$i/||" )
             )
+            [[ ${#COMPREPLY[@]} == 1 && ${COMPREPLY[0]} == *.class ]] && \
+                compopt +o nospace &>/dev/null
 
             # FIXME: if we have foo.class and foo/, the completion
             # returns "foo/"... how to give precedence to files

test/lib/completions/java.exp

@@ -18,7 +18,7 @@ assert_complete_any "java -"
 sync_after_int
 
 
-assert_complete "b bashcomp.jarred toplevel" "java "
+assert_complete "b bashcomp.jarred c. toplevel" "java "
 
 
 sync_after_int