<div><br><div class="gmail_quote">On Fri, Jul 22, 2011 at 7:51 AM, Mirko Wischer <span dir="ltr">&lt;<a href="mailto:Mirko.Wischer@sma.de">Mirko.Wischer@sma.de</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>
I tried to make a patch that fixes the problem - see attached file.<br>
My example code now works without problems. I also attached<br>
the patch to novell-mono-bugzilla, but I&#39;m not really sure if that&#39;s<br>
the right place.<br>
<div class="im"><br>
Bye Mirko</div></blockquote><div><br></div><meta charset="utf-8">You don&#39;t need the second check. The assert is there to make sure we have everything in sync.<div><br></div><div>I believe a nicer fix would be to do something like &quot; if (!ins-&gt;next) { ins-&gt;next = OP_NOP; break; }&quot; only</div>
<div>as it would reduce the diff size and nesting levels.</div><div><br></div><div>The best way to get the fix in is to fork mono from github and make a pull request. Reference the bug number on it</div><div>and we&#39;ll take care of the rest.</div>
<div><br></div><div>Thanks a lot for working on this.</div><div><br></div><div>Rodrigo</div></div></div>