[cfe-dev] CIndex changes

[steve naroff]

On Oct 27, 2009, at 7:59 PM, Ted Kremenek wrote:

> Thanks John.  The patch makes it very clear.
>
> The problem is that we don't want to add static data to CIndex, as  
> we want the functions to be as reentrant as possible.
>
> Steve/Doug: I believe you guys discussed this one.  Comments?
>

Hey Ted,

We haven't discussed it yet. I expect to speak with Doug today/ 
tomorrow and make a decision by the end of the week.

snaroff

> On Oct 27, 2009, at 2:51 PM, John Thompson wrote:
>
>> Thanks, Ted.
>>
>> I'm sorry I've been disconnected from this.  What particular code  
>> were you looking at that returned a 'const char*' that referred to  
>> the buffer associated with a temporary std::string?
>> It's in clang_getTranslationUnitSpelling, clang_getDeclSpelling,  
>> and clang_getCursorSpelling.  The enclosed patch with my hacky work- 
>> around will show the specific places.
>>
>> I also moved getLocationFromCursor to fix a VC++ message about the  
>> return type not being permitted inside an extern "C" {} block.
>> -John
>>
>> -- 
>> John Thompson
>> John.Thompson.JTSoftware at gmail.com
>>
>> <cindex_strings.patch>_______________________________________________
>> cfe-dev mailing list
>> cfe-dev at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20091028/7a7e909d/attachment.html>