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.

-- Sean Silva


On Fri, Nov 8, 2013 at 3:57 PM, Johan Engelen <[email protected]>wrote:

> Hi all,
>   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.
> 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.
> I tested the change and the patched plugin works in both Sublime Text 2
> and 3.
>
> Thanks for the review, (and commit ;)
>   Johan
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to