<BLOCKQUOTE style="PADDING-LEFT: 5px; MARGIN-LEFT: 5px; BORDER-LEFT: #ff0000 2px solid">
<P>> Message du 21/01/05 04:10<BR>> <BR>> ChangeLog:<BR>> Ulrich Czekalla <ULRICH@CODEWEAVERS.COM><BR>> * add support to interrogate gdi objects<BR>several comments:</P>
<P>* the changes in db_disasm.c are not declared in the changelog. Morever, I don't see the need of passing an extra parameter in db_task_printsym as you don't need it (maybe you forgot that part from the patch ?)</P>
<P>* for the pure gdi info.</P>
<P>- always use (when looking for a symbol) the name of the DLL you're looking the symbol from. this has several good side effects: first, it shortens the scope for searching (as dbghelp treats all global symbol - static or public - in the same name space), this reduces the likely hood of name conflicts. Secondly, the search will be much faster (as dbghelp won't be searched for all loaded DLLs), and won't force loading of symbols in DLLs for which we haven't looked up symbols yet (deferred symbol loading). For example, you should use gdi32!GDI_HeapSel for the symbol out of gdi32.</P>
<P>* why are you using the ldt_copy at all ? there's support in winedbg for linearizing 16bit addresses, so making a 16 bit addr out of gdi-sel and handle and linearizing it should work.</P>
<P>* you should add the same functionality to the gdbproxy (or at least add a TODO in winedbg.c to remain that it has to be done)</P>
<P>* you should document the feature (info.c and documentation/debugger.sgml)</P>
<P>A+</P></BLOCKQUOTE>