Bo Peng wrote: >> I haven't been able to cause a problem after deleting these lines. The >> reason is that the InsetInclude::latex() routine just ignores the >> options, anyway, if we're not dealing with a listings include. The >> advantage to deleting them turns out to be that if you switch from >> "listings" to "verbatim" and then go back, your options re-appear. > I have not checked in details but the original idea is that when > option is set for listings, it remains there if we switch to another > type. This may or may not confuse users. Also, setOption may (not > sure) be called to set options for other types, although they will not > be used eventually. > Because the 're-appearing' option seems to be a good thing to have, > you can remove them if you can not see any bad side-effect. I've committed this. By the way, another problem I noticed with the QInclude dialog: If you enter a parameter in the caption field (say) and hit return, it's not recorded. Should be a simple matter, but I'm on the trail of other beasties.
Richard -- ================================================================== Richard G Heck, Jr Professor of Philosophy Brown University http://frege.brown.edu/heck/ ================================================================== Get my public key from http://sks.keyserver.penguin.de Hash: 0x1DE91F1E66FFBDEC Learn how to sign your email using Thunderbird and GnuPG at: http://dudu.dyn.2-h.org/nist/gpg-enigmail-howto
