[Therion] therion 5.3.16

Wookey wookey at wookware.org
Thu Jan 8 00:44:54 CET 2015


+++ Olly Betts [2015-01-07 20:29 +0000]:
> On Wed, Jan 07, 2015 at 09:11:06AM +0000, Wookey wrote:
> > +++ Stacho Mudrak [2014-12-30 11:09 +0100]:
> > >      * 82-nolang-segfault-fix.patch  (this avoids a genuine error case of
> > >      language not being defined)
> > > 
> > >    I believe, this error is already fixed. Are you able to reproduce it with
> > >    current version of therion?
> > 
> > I can't remember what the issue was or how to reproduce it. Anyone remember?
> 
> No, but thlang_getid() now starts with an explicit check to handle the
> case where the id is -1, so the patch is clearly no longer needed:
> 
>     const char * thlang_getid(int id) {
>       if (id < 0) {
> 	return "en";
>       }

> The important conclusion is that it clearly isn't useful for 5.3.16.

Agreed. That's good enough for me. therion 5.3.16-4 uploaded just now
with the offending patch removed.

Wookey
-- 
Principal hats:  Linaro, Debian, Wookware, ARM
http://wookware.org/



More information about the Therion mailing list