Re: [PATCH] bug 153, 1066: Fix search in bookmark manager.

> test_search() was supposed to compare bookmark titles with
> strcasestr(), but in commit 311d95358de258656a0c7145a2dcff943fcdcd4b
> "bug 153, 1066: Convert bookmarks to/from UTF-8 when searching."
> on 2009-02-08, I inadvertently changed that to strcasecmp(), even
> while adding a comment about why strcasestr() is needed.  strcasestr()
> returns non-NULL if the strings match, and strcasecmp() returns
> nonzero if they differ, so the search didn't work at all.
> ---
> Stephen Stocker <lpar at terraworld.net> writes:
> 
> >    Since sometime before Feb 14, 2009, the bookmark search feature seems to be
> > broken in the daily snapshots. If I hit "s" to display the bookmarks dialog,
> > then "s" again to search, and type in whatever I'm looking for, it moves to the
> > next item in the bookmarks. Searching again moves it ahead one item again, and
> > so on. In other words, everything appears to match the search.
> 
> Thanks for reporting this.  It was my fault.
> I have pushed the following fix to elinks-0.12 now.
> It can be cleanly merged to master.  I didn't push the merge.
> 
> src/bookmarks/dialogs.c |    4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/bookmarks/dialogs.c b/src/bookmarks/dialogs.c
> index 69340b1..b20815c 100644
> --- a/src/bookmarks/dialogs.c
> +++ b/src/bookmarks/dialogs.c
> @@ -687,8 +687,8 @@ test_search(struct listbox_item *item, void *data_, int *offset)
> }
> 
> if (title) {
> -                               ctx->found = strcasecmp(title,
> -                                                       ctx->title);
> +                               ctx->found = (strcasestr(title, ctx->title)
> +                                             != NULL);
> mem_free(title);
> }
> /** @todo Tell the user that the string could
> --
> 1.6.0.6.4.g22c5

   Thanks! This fixed it.

   Take care,
   Steve


-- 
Hope, Despair and the Triumph of Life
http://www.hdtl.org
Abuse Prevention/Survivor Support
_______________________________________________
elinks-users mailing list
[email protected]
http://linuxfromscratch.org/mailman/listinfo/elinks-users