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

["Marcus Fritzsch" <fritschy googlemail com>]

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?

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

Marcus