* 2010-09-03 Helmut Eller heller@common-lisp.net | For C-c C-k, ask before loading possibly broken fasl files. | | * slime.el (slime-compilation-result): Add 2 slots: loadp and | faslfile. | (slime-compilation-finished): Use them to load the faslfile. | * swank.lisp (:compilation-result): Add 2 slots. Use keyword | constructor.
when (slime-compile-and-load-file) notices harmless Warnings during compilation, it prompts with (y-or-n-p "Compilation failed. Load fasl file anyway? ")
There is no way for the user to scroll the *slime-compilation* window to see if the warnings are harmless or not, before he can answer `y'
I would suggest giving the user an option like like in the appended patch, but I suspect Helmut would reject it as it would complicate slime, and would find it simpler to add a new keymap for this situation, with new functions to scroll the buffer and also remap [next-line] and [previous-line] to the new functions so any user level override without modifying slime.el is impossible.
[BTW on an unrelated note, C-h k C-c C-k in slime buffers in (Emacs git 2010-10-03) says: C-c C-c is undefined. The binding works but doesn't show up in C-h b either. I'm afraid the baroquely over-engineered keybinding mechanisms makes it impossible to debug where the problem is.]
-- Madhu
+(defvar slime-compilation-force-load t) + (defun slime-compilation-finished (result) (with-struct (slime-compilation-result. notes duration successp loadp faslfile) result @@ -2673,7 +2675,7 @@ to it depending on its sign." (slime-highlight-notes notes)) (run-hook-with-args 'slime-compilation-finished-hook notes) (when (and loadp faslfile - (or successp + (or successp slime-compilation-force-load (y-or-n-p "Compilation failed. Load fasl file anyway? "))) (slime-eval-async `(swank:load-file ,faslfile)))))