commit: Improve style and add tests for 'built-in-symbol-file'.

Stephen Turnbull writes: It turns out that although Aidan's test !NILP does not have a multiple evaluation problem, at least in some configurations SUBRP does. So it's a bad idea to have something like (fun = Fcdr_safe (fun)) as a macro argument, and it would be especially bad if somebody decided to make the "obvious" optimization of moving the assignment into the SUBRP invocation and eliminate the !NILP check altogether in that way. I probably would have left it alone, but I was curious so wrote some tests and decided to add them to the test suite. _______________________________________________ XEmacs-Patches mailing list XEmacs-Patches(a)xemacs.org http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-patches