<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
On 5/16/2012 14:21, Jacek Caban wrote:
<blockquote cite="mid:4FB38D9E.70804@codeweavers.com" type="cite">
<pre wrap="">---
dlls/ieframe/Makefile.in | 1 +
dlls/ieframe/dochost.c | 16 ++-
dlls/ieframe/ieframe.h | 2 +
dlls/ieframe/shelluihelper.c | 378
++++++++++++++++++++++++++++++++++++++++++
4 files changed, 395 insertions(+), 2 deletions(-)
create mode 100644 dlls/ieframe/shelluihelper.c
</pre>
</blockquote>
<blockquote type="cite">
<pre wrap="">+ <b class="moz-txt-star"><span class="moz-txt-tag">*</span>ppDispatch = (IDispatch<span class="moz-txt-tag">*</span></b>)This->shell_ui_helper;
+ IDispatch_AddRef(*ppDispatch);
+ return S_OK;</pre>
</blockquote>
IShellUIHelper2 supports IDispatch, so these three lines and a cast
could be replaced with one QI like:<br>
<pre wrap="">return IShellUIHelper2_QueryInterface(This->shell_ui_helper, &IID_IDispatch, (void**)ppDispatch);
if I'm not missing something.
</pre>
<br>
</body>
</html>