<div>Hi,</div><div><br></div><div>Your patch looks good, have you pushed it?</div><div> - Otherwise please do, my LibreOffice clone isn&#39;t up to date at the moment :(</div><div><br></div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204, 204, 204);border-left-style:solid;padding-left:1ex">


in case my patch would just hide a deeper problem.</blockquote><div>I don&#39;t think it hides a deeper problem (except lack of comments :) ), as far as I remember there&#39;re other places where pViewShell-&gt;GetEditWindow() is NULL tested. Also the method explicity returns NULL when the edit window doesn&#39;t exists (I&#39;m guessing either not created yet, or disabled).</div>


<div> - So I think it&#39;s okay.</div><div><br></div><div>By the way, nice to see that you&#39;re working GSoC again this year.</div><br clear="all">--<br>Regards Jonas Finnemann Jensen.<br>
<br><br><div class="gmail_quote">On Thu, Jul 14, 2011 at 16:50, Miklos Vajna <span dir="ltr">&lt;<a href="mailto:vmiklos@frugalware.org" target="_blank">vmiklos@frugalware.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Hi,<br>
<br>
When I try to open<br>
<a href="http://people.freedesktop.org/~vmiklos/math.word2007.docx" target="_blank">http://people.freedesktop.org/~vmiklos/math.word2007.docx</a> on master<br>
(with experimanl features disabled, so the inline formula editor is<br>
disabled), double-click on the math formula, Math opens, finally I click<br>
on the formula, Math crashes.<br>
<br>
The attached patch fixes the problem here, but I&#39;m sending it for review<br>
as I&#39;m totally unfamiliar with the starmath code - in case my patch<br>
would just hide a deeper problem.<br>
<br>
Thanks.<br>
</blockquote></div><br>