<br><br><div class="gmail_quote">2009/11/12 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><br>
> > +  then<br>
> > +    AC_ARG_WITH(sqlite,<br>
> > +      [    --with-sqlite=PATH sqlite path linking])<br>
> ><br>
> > Do we really need this? Also, same error as AC_ARG_ENABLE.<br>
><br>
> Yes, because Darwin doesn't have the latest sqlite.<br>
<br>
</div>What's wrong with contribs? Why do you need your own path?<br>
<div><br></div></blockquote><div><br></div><div>We can include sqlite in the contribs.</div><div>MacPorts had an old version of sqlite and a new version was required for testing. So, with-sqlite was included. Shall I remove it and include sqlite in the contribs? Or do we want to keep the contribs minimal?</div>

<div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div> </div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
> > +static char* VMSprintf( const char* psz_fmt, va_list args )<br>
> > +{<br>
> > +    char *psz = sqlite3_vmprintf( psz_fmt, args );<br>
> > +    char *ret = strdup( psz );<br>
> > +    sqlite3_free( psz );<br>
> > +    return ret;<br>
> > +}<br>
> ><br>
> > Do we really need this double copy/allocation?<br>
><br>
> Yes. psz is allocated with sqlite3_malloc. Some "fastmem" builds can use<br>
> different allocation strategies, hence we cannot rely on free() to free<br>
>  psz. Hence the copy.<br>
<br>
</div>That's not the point. Why do you need strdup and free?<br></blockquote><div>Is there a way to do this without strdup?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><br>
> > Same stalling problem.<br>
> ><br>
> > IMHO, RollbackTransaction should return void, because it does not make<br>
> > much sense to fail here.<br>
><br>
> I agree that it should return void. However, the people who made sqlite<br>
> decided that a ROLLBACK can fail if there are pending reads/writes.<br>
<br>
</div>I don't care what the SQLite people decided. There _must_ be a way to recover<br>
from the error. Keeping a stalled mutex is _not_ acceptable under any<br>
circumstances. In fact, it will cause an assertion failure when you close the<br>
DB.<br>
<font color="#888888"><br></font></blockquote><div>Ah. The Rollback needs to be called again after a while.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<font color="#888888">
--<br>
</font><div>Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" target="_blank">http://www.remlab.net/</a><br>
</div><div><div></div><div>_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Regards,<br>Srikanth Raju<br>