<div dir="ltr">Thanks. Committed as r194287.<div><div><br></div><div>-- Sean Silva</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 8, 2013 at 5:07 PM, Johan Engelen <span dir="ltr"><<a href="mailto:jbc.engelen@swissonline.ch" target="_blank">jbc.engelen@swissonline.ch</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    Thanks for the feedback. I have attached the improved patch.<br>
    <br>
    Thanks,<br>
      Johan<div><div class="h5"><br>
    <br>
    <br>
    <div>On 8-11-2013 22:25, Sean Silva wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">Your patch doesn't do what you think. In Python 2,
        it is still invoking a print statement with the expression
        `(error)` which happens to evaluate to just `error`. You need a
        `from __future__ import print_function`, which will make it be a
        function call in both Python 2 and 3.
        <div>
          <div><br>
          </div>
          <div>-- Sean Silva</div>
        </div>
      </div>
      <div class="gmail_extra"><br>
        <br>
        <div class="gmail_quote">On Fri, Nov 8, 2013 at 3:57 PM, Johan
          Engelen <span dir="ltr"><<a href="mailto:jbc.engelen@swissonline.ch" target="_blank">jbc.engelen@swissonline.ch</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
              Sublime Text 3 uses Python 3.3 for its plugins (instead of
            2.6 that Sublime Text 2 uses). The attached patch fixes
            clang-format-sublime.py to work in Sublime Text 3.<br>
            I am not a Python expert, but it seems for this plugin, the
            only relevant change is that "print" became a function, so
            parens are needed.<br>
            I tested the change and the patched plugin works in both
            Sublime Text 2 and 3.<br>
            <br>
            Thanks for the review, (and commit ;)<span><font color="#888888"><br>
                  Johan<br>
              </font></span><br>
            _______________________________________________<br>
            cfe-commits mailing list<br>
            <a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
            <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
            <br>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </div></div></div>

</blockquote></div><br></div>