[cfe-commits] [PATCH] [cindex.py] Do not fail when registering non-existing functions

[Tobias Grosser]

On 08/31/2012 10:43 AM, Tobias Grosser wrote:
> On 08/27/2012 01:07 AM, Tobias Grosser wrote:
>> On 08/25/2012 08:45 AM, Gregory Szorc wrote:
>>> I agree with the sentiment of wanting increased compatibility, but I'm
>>> hesitant that this change is the right answer.
>>
>> Hi Gregory,
>> thanks for the feedback.
>
> [..] (A long reply, discussing clang compatibility strategies)
>
> I updated the patch according to your comments. The patch does now
> provide an informative error message, if a method cannot be found in
> libclang.so:
>
> -----------------------------------------------------------------------
> [cindex.py] Add helper to initialize libclang functions
>
> The helper allows us to define how the initialization of functions
> should behave. We use this patch to provide an informative error
> message, in case a function is not available:
>
> "LibclangError: /home/grosser/Projekte/llvm/install/lib/libclang.so:
> undefined symbol: clang_method_added_in_2020. Please ensure that your
> python bindings are compatible with your libclang.so version."
>
> This patch also ensures that no spelling mistakes slip into the library
> initialization. At the moment, there are a couple of 'argtype' ->
> 'argtypes' mispellings that have been overlooked.
> -----------------------------------------------------------------------

And here the patch, that does not include the typo that I used to test 
that feature.

Tobi

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-cindex.py-Add-helper-to-initialize-libclang-function.patch
Type: text/x-patch
Size: 29415 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120831/b01e2d7f/attachment.bin>