Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

emacs: errors about auto-complete #15

Closed
oandrieu opened this issue Feb 21, 2013 · 1 comment
Closed

emacs: errors about auto-complete #15

oandrieu opened this issue Feb 21, 2013 · 1 comment

Comments

@oandrieu
Copy link
Contributor

I do not have the auto-complete package and merlin.el complains a bit:

--- /home/oandrieu/Downloads/merlin.el  2013-02-21 11:16:47.391076941 +0100
+++ /home/oandrieu/.opam/system/share/emacs/site-lisp/merlin.el 2013-02-21 11:09:54.597652168 +0100
@@ -419,7 +419,8 @@
     (requires . 3)
     ))

-(ac-define-source "merlin" merlin-ac-source)
+(when (featurep 'auto-complete)
+  (ac-define-source "merlin" merlin-ac-source))

 ;; Get the type of an element"
 (defun merlin-trim (s)
@@ -584,7 +585,8 @@
   (interactive)
   (progn
     (merlin-start-process)
-    (auto-complete-mode)
+    (when (featurep 'auto-complete)
+      (auto-complete-mode))
     (set (make-local-variable 'merlin-lock-point) (point-min))
     (set (make-local-variable 'merlin-buffer) nil)
     (set (make-local-variable 'merlin-result) nil)
@ghost ghost assigned asmanur Feb 21, 2013
asmanur added a commit that referenced this issue Feb 21, 2013
@asmanur
Copy link
Contributor

asmanur commented Feb 21, 2013

Hi,

Thanks for the patch, it is available in master.

@asmanur asmanur closed this as completed Feb 21, 2013
catern pushed a commit to catern/merlin that referenced this issue May 6, 2024
…w-files

Import script pulls in new files added in flambda-backend
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants