I spent some time recently adding more/improving the visualizers, but I wasn't focused on any in particular.  I was mostly just working my way through objects in the debugger and improving what I encountered.  I have a pretty decent grasp at this point of what the expression language can do, after fighting with it for several hours.  If there are Types that more experienced devs would like to see visualized, I'd be happy to take a stab at them.<br>
<br>Unfortunately, I got pulled away from the task for the time being, so here's what I had when I stopped.<br><br>Can we get this committed as well?<br><br>Thanks,<br><br>Jay Blanchard<br><br><div class="gmail_quote">
On Wed, May 16, 2012 at 6:31 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, May 16, 2012 at 2:31 PM, Will Wilson <<a href="mailto:will@indefiant.com">will@indefiant.com</a>> wrote:<br>
> On 3 May 2012 19:40, Will Wilson <<a href="mailto:will@indefiant.com">will@indefiant.com</a>> wrote:<br>
>>>>>> For Visual Studio users: Here's little patch to the<br>
>>>>>> clangVisualizers.txt that allows the contents of a SmallString to be<br>
>>>>>> seen in string form, rather than (somewhat less usefully) as a<br>
>>>>>> SmallVector.<br>
>>>>><br>
>>>>> I assume this wouldn't work for SmallStrings that include embedded<br>
>>>>> '\0' characters? Though I suppose we don't do any better for StringRef<br>
>>>>> either, so perhaps that's not a priority.<br>
>>>><br>
>>>> Unfortunately not, I can't find any support for anything other than<br>
>>>> basic zero-terminated strings in the syntax.<br>
>>><br>
>>> I don't know anything about VS, really, but SmallStrings are not usually null-terminated. Have you tested this when there is definitely something after the string?<br>
>><br>
>> Yes, in such cases you see whatever follows the string data. Therefore<br>
>> (as with StringRef) you need to be careful interpreting the string<br>
>> data in the debugger preview. That said, I'd far rather have the<br>
>> utility of seeing it displayed as a string with garbage at the end<br>
>> than have to interpret it as an array of characters. And either way,<br>
>> you may still view it as an array by expanding the string node to show<br>
>> the underlying view of the SmallVector<> which also lets you see the<br>
>> true extent of it.<br>
>><br>
>> So it's definitely imperfect, but better than nothing :)<br>
><br>
> Any thoughts on getting this committed? It's pretty minor but helpful.<br>
<br>
</div>Sorry, just lost track of it. Committed as r156959.<br>
<div class="im"><br>
> Incidentally, I could look at writing a custom debug plugin in the<br>
> future to deal better with StringRef and it's relatives in VS.<br>
<br>
</div>Sounds good - I don't personally use VS for development, but happy to<br>
throw these in.<br>
<br>
[maybe I mentioned this already & it was refuted, but: It'd be nice if<br>
they could use some common functionality (it's weird that visualizing<br>
a derived class can't use a base class's visualization behavior by<br>
default... but perhaps that makes sense, I'm not sure - if not by<br>
default, perhaps explicitly just by casting to the base)]<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">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>
</div></div></blockquote></div><br>