<br><br><div class="gmail_quote">On 10 March 2010 04:39, Nick Treleaven <span dir="ltr"><<a href="mailto:nick.treleaven@btinternet.com">nick.treleaven@btinternet.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Tue, 9 Mar 2010 19:48:39 +1100<br>
Lex Trotman <<a href="mailto:elextr@gmail.com">elextr@gmail.com</a>> wrote:<br>
<br>
> > ft is NULL there, which looks like a valid value, since ft is tested for<br>
> > NULL explicitly in line 1878. So I propose the following patch.<br>
</div>...<br>
<div class="im"><br>
> Thanks, one more change is needed, ft isn't initialised to NULL so on my<br>
> machine it doesn't crash :-S, needs the declaration of ft to be initialised<br>
> to NULL so your test will reliably fail if ft isn't set.<br>
><br>
> I will have to get a clean copy of trunk to apply the patch to (mine is full<br>
> of changes :-) but will do so as soon as possible if no one beats me to it.<br>
<br>
</div>Oops, doc->file_type is never NULL now so I assumed ft never was.<br></blockquote><div><br>Same here, but of course doc can be null dammit!<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<br>
BTW is a separate ft argument necessary if we have doc?<br></blockquote><div><br>No, originally it didn't use doc just ft, evolving software...<br><br>Cheers<br>Lex <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<br>
Regards,<br>
<font color="#888888">Nick<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de">Geany-devel@uvena.de</a><br>
<a href="http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
</div></div></blockquote></div><br>