<p dir="ltr">I don't have commit access, so please go ahead.</p>
<p dir="ltr">Wilfred</p>
<div class="gmail_quote">On 6 Jan 2015 23:45, "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">btw, do you have commit access or should I commit it?<br>
<br>
On 6 January 2015 at 11:22, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> LGTM, thanks!<br>
><br>
> On 4 January 2015 at 19:08, Wilfred Hughes <<a href="mailto:me@wilfred.me.uk">me@wilfred.me.uk</a>> wrote:<br>
>> Hi<br>
>><br>
>> I'd like users to be able to install the LLVM major modes from<br>
>> MELPA[1], the most popular Emacs package repository. However, these<br>
>> files are not well-formed packages.<br>
>><br>
>> I've made the following changes:<br>
>><br>
>> * Both files have valid package headers and footers (you can verify<br>
>> with M-x checkdoc).<br>
>> * Fixed style warnings generated by checkdoc.<br>
>> * Fixed a byte-compiler warning in llvm-mode.el.<br>
>> * Ensure that the modes are autoloaded, so users do not need to<br>
>> (require 'llvm-mode) to use them.<br>
>><br>
>> Patch attached, let me know what you think.<br>
>><br>
>> Thanks<br>
>> Wilfred<br>
>><br>
>> [1] <a href="http://melpa.org/" target="_blank">http://melpa.org/</a><br>
>><br>
>> _______________________________________________<br>
>> LLVM Developers mailing list<br>
>> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
>><br>
</blockquote></div>