[cfe-commits] [PATCH 4/4] [clang.py] TranslationUnit API improvements
anders at 0x63.nu
Mon May 14 12:23:30 PDT 2012
On Sat, May 12, 2012 at 02:19:08PM -0700, Gregory Szorc wrote:
> On Mon, May 7, 2012 at 11:17 PM, Anders Waldenborg <anders at 0x63.nu> wrote:
> > Also possibly mention the -x argument in the paragraph
> > about "C++ in test.c".
> What are you referring to?
I'm was referring last paragraph in TranslationUnit.from_source
Also note that Clang infers the source language from the extension of
the input filename. If you pass in source code containing a C++ class
declaration with the filename "test.c" parsing will fail.
I was thinking that it would be useful to mention "-x" in that
TranslationUnit.from_source(args=["-x", "c", "filewithnonstandardextension.code"])
> > Did you consider to make the different types of
> > TranslationUnitSaveError subclasses?
> Yes, and it didn't feel very Pythonic. If I were writing Java, this is
> probably how I would do it. But, Python is not Java. If there is more
> data hung off save errors later, then we have a case for different
> classes. Until then, I think simplicity should win.
<Bikeshedding begins - feel free to ignore>
I'm not sure I agree about which is most pythonic of those variants.
I'm not dutch but I would say the most pythonic way would be to use
built-in exceptions to greatest possible extent (so in this case if
res == 1: raise RuntimeError("Known unknown error saving tu") elif res
== 2: raise ValueError("Error saving tu with errors") elif res == 3:
raise TypeError("Not a valid tu")).
My point was not to make a javaish huge class hierachy just because.
But as the only attribute on the exception was a sub-type it felt
natural to instead expose this in the type.
This gives effortless string representation (as the class name is
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "<stdin>", line 2, in save
TranslationUnitSaveWithErrorsError: Couldn't save translationunit
Also I find it annoying to have to inspect attributes of the exception
to know how to handle it. Like this:
except IOError as e:
if e.errno in nonfatalerrnos:
This became way too long for such tiny detail. I guess I just had to
defend myself after being accused for writing java in python :)
More information about the cfe-commits