Skip to content
This repository was archived by the owner on Feb 22, 2021. It is now read-only.

Conversation

syohex
Copy link

@syohex syohex commented Nov 25, 2015

There are following byte-compile warnings about this.

In rtog/pass-code:                                                                     
repl-toggle.el:134:24:Warning: `4' is a malformed function                             
repl-toggle.el:134:24:Warning: `16' is a malformed function                            
repl-toggle.el:134:24:Warning: `64' is a malformed function                            

In end of data:                                                                        
repl-toggle.el:280:1:Warning: the function `case' is not known to be defined.          

@tomterl
Copy link
Owner

tomterl commented Jan 17, 2016

Which version of emacs are you using? I don't get these warnings -- and AFAIR cl-lib handling changed considerably in newer emacsen -- but I can't remember which version introduced the change.
So I guess the change would require version checks etc.

@syohex
Copy link
Author

syohex commented Jan 17, 2016

Which version of emacs are you using?

Emacs 24.3 and Emacs 24.5.

@syohex
Copy link
Author

syohex commented Jan 17, 2016

At lease, you should add following code.

(eval-when-compile
  (require 'cl))

tomterl added a commit that referenced this pull request Jan 17, 2016
@tomterl
Copy link
Owner

tomterl commented Jan 17, 2016

Done in master -- is this sufficient for you?

@syohex
Copy link
Author

syohex commented Jan 17, 2016

Yes.

@syohex syohex closed this Jan 17, 2016
@syohex syohex deleted the cl-lib branch January 17, 2016 13:34
@tomterl
Copy link
Owner

tomterl commented Jan 17, 2016

Cool - thanks for caring and your time -- with small packages like these one never knows they're used until something like this happens.

If you're on melpa-stable, a new release is imminent once #7 is finished.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants