Andrew Hyatt writes: > Paul Kinnucan <[EMAIL PROTECTED]> writes: > > > [EMAIL PROTECTED] writes: > > > Update of /pack/anoncvs/jde/lisp > > > In directory sunsite.dk:/tmp/cvs-serv11659 > > > > > > Modified Files: > > > jde-xref.el > > > Log Message: > > > Made jde-xref-store-prefixes non-optional. I think if we didn't do > > > this, most people would leave it blank, therefore significantly > > > impacting both size of the database and the time and memory it takes > > > to make it. > > > > > > > Hi Andy, > > > > How about making nil default to the package of the class in the > > current source buffer? >
I think it's nice to have all the jde-xref variables to have defaults that allow a new user to try it out quickly without having to do any preliminary setup, which is now required with your latest change. Previously, the default for this variable was to cross-ref the universe by default. I'm suggesting that we change it to cross-ref the current package. Is this what the user is likely to want ultimately? No, but it's a good starting point. The new user should quickly notice that the xrefing is limited and this should lead them to the doc to find out why. > I'm not sure I want to make any assumptions. Typically, people would > want their database to encompass more than one package. Plus, I > anticipate that this will not always be invoked from a Java buffer. > For instance, you could rebuild the database every night, and in that > case you wouldn't be in a buffer. > > > > > Also, I'd like to suggest renaming jde-xref-store-prefixes to > > jde-xref-packages as that more clearly indicates the variables > > purpose. > > > > Let me think about this one. The thing is, the variable doesn't > contain package names, it contains package prefixes. What's the difference? > But the name is a bit confusing, I agree. I think this variable, whatever its name, should allow the user to specify the packages that the user wants to xref, including specifying that all subpackages of a package be xrefed, e.g., if I have a package structure like this: pkgA pkgAa pkgAb I should be able to specify pkgA and have everything in pkgA, including pkgAa and pkgAb, xrefed. On the other hand, if I specified pkgA.pkgAb, I'd expect only pkgAb to be xrefed. - Paul
