<div dir="ltr">(FWIW you don't _have_ to rename the script, you could use `clang_format = __import__("clang-format"))</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 16, 2013 at 10:14 AM, Steffen Prohaska <span dir="ltr"><<a href="mailto:prohaska@zib.de" target="_blank">prohaska@zib.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
On Sep 16, 2013, at 6:04 PM, Tobias Grosser <<a href="mailto:tobias@grosser.es">tobias@grosser.es</a>> wrote:<br>
<br>
> Steffen, I believe the idea of your patch is great. Unfortunately it has from my point two issues:<br>
><br>
> 1) You rely on features of very recent vim versions<br>
<br>
</div>I've addressed this in in the revised script that I've just sent in a separate mail.<br>
<div class="im"><br>
<br>
> As this is not a complicated plugin, there seems no reason to drop support for older vim versions.<br>
><br>
> 2) You require python in the .vimrc file<br>
><br>
> I would prefer to not require python in the .vimrc, but have the plugin report an error when it is called without python support.<br>
<br>
</div>Python has also been required by the previous version of the script.  So my proposed changes don't make the situation worse.<br>
<br>
I agree that a full plugin would be ideal.  I think that it would ideally be hosted in a separate git repo on Github, so that Pathogen or Vundle can be used to install it.  However, I don't plan to create such a plugin anytime soon.<br>

<span class="HOEnZb"><font color="#888888"><br>
        Steffen<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div>