<P>I don't think this patch should go in as it is:</P>
<P>- I don't think it's a good idea to have two ways to express a pointer type in the debugger (one being the standard dbghelp's SymTagPointerType, the other one the new deref count you've introduced). It would be better to add dynamic winedbg types (in the contrary to the static one's we already use dbg_itype_*), when we cannot find one in a module.</P>
<P>- the returned string (for internal types' name) should be HeapAllocate:d, not a static buffer</P>
<P>- please, use the current style for parenthesis, braces... and don't add a new one<BR><BR><BR>A+<BR><BR></P>
<BLOCKQUOTE style="PADDING-LEFT: 5px; MARGIN-LEFT: 5px; BORDER-LEFT: #ff0000 2px solid">> Message du 01/03/05 18:28<BR>> De : "Oliver Stieber" <OLIVER_STIEBER@YAHOO.CO.UK><BR>> A : wine-patches@winehq.com<BR>> Copie � : <BR>> Objet : [winedbg] Get expressions working again.<BR>> <BR>> Hi,<BR>> This patch starts to get expressions working again and<BR>> also allows derefrencing where there is no explicit<BR>> type for that dereference. e.g. <BR>> <BR>> (short)0x12356 now returns the correct value instead<BR>> of an error<BR>> <BR>> *((int *)0x23467) is now a valid expression.<BR>> and <BR>> ***((int ***)0x23467) should also work properly.<BR>> <BR>> This should also fix watches with expressions but I<BR>> haven't given it much testing.<BR>> <BR>> Send instant messages to your online friends http://uk.messenger.yahoo.com ><BR>> [ winedbg_expressions_os_1.patch (18.4 Ko) ]</BLOCKQUOTE>