Re: [Tracker] [PATCH] Potential crasher in set_language()

[jamie <jamiemcc blueyonder co uk>]

On Mon, 2007-07-23 at 00:55 +0200, Marcus Fritzsch wrote:
> On 7/23/07, jamie <jamiemcc blueyonder co uk> wrote:
> > On Mon, 2007-07-23 at 00:16 +0200, Laurent Aguerreche wrote:
> > > Le lundi 23 juillet 2007 Ã 00:13 +0200, Laurent Aguerreche a Ãcrit :
> > > > Le dimanche 22 juillet 2007 Ã 22:33 +0100, jamie a Ãcrit :
> > > > > On Sun, 2007-07-22 at 23:21 +0200, Laurent Aguerreche wrote:
> > > > > > Hello,
> > > > > >
> > > > > > a patch to fix a potential crasher in tracker_set_language() function :
> > > > > >
> > > > > > - g_free() was called whether pointer to free could be NULL;
> > > > > > - we were mixing allocated strings and on-stack strings;
> > > > > > - added tracker_get_english_lang_code() function and put back tmap
> > > > > > implementation in tracker-utils.c (avoid warnings at building time)
> > > > > >
> > > > > > + cosmetic changes : I removed some empty lines for instance.
> > > > > >
> > > > > >
> > > > > > Can I commit?
> > > > >
> > > > > ok for this one but no more cleanups pls until post 0.6 as I want to
> > > > > release tomorrow
> > > > >
> > > > > (I would prefer bugfixes to be separate from wide ranging cleanups if
> > > > > possible)
> > > >
> > > > Ok.
> > > >
> > > >
> > > > About Gaim: it recently changed its name in favour of Pidgin. Gaim uses
> > > > library Purple as a network layer. This library is also responsible for
> > > > chat logging.
> > >
> > > Sorry, it is Pidgin which uses Purple.
> > >
> > >
> > > > For people who used Gaim (like me), "logs" directory in "~/.gaim/logs"
> > > > became a symbolic link to "~/.purple/logs".
> > > > Since trackerd do not follow links, it does not see "logs" link as a
> > > > correct directory and do not index anything from this directory.
> > > > Furthermore, we need to index Purple chat logs (not Pigin directly
> > > > because Purple may be used by others programs).
> > > >
> > > > How to fix that? Will we delay that for post Tracker 0.6?
> >
> >
> > I quickly added ~/.purple/logs as a gaim conversation directory
> >
> > I need someone to test its picking then up though
>
> Seems to work nice, files indexed, searching works.
>
> Issues (maybe):
>
> * Did anybody noticed that highlighting of snippet-words (i.e. the
> stemmed search words?) does not work?

it works for me

> * Chat logs are not 'accumulated' to 'All Files' in the sidepane of
> t-s-t - is this correct?

yes chat logs are separate from files in tracker

> Marcus